Yes, this implementation does authenticate and work. However, I was having an issue where occasionally, when I have been watching a feed for awhile and back out to open another app and then return to SN to watch a channel, it'll come up 'Unable to authorise channel'. This is easily fixed by backing out of the channel listing and reopening the addon to re-auth, but may be something the typical user would have an issue with. I haven't had time to explore it further and analyze the logs, but plan to.

I have been using it personally though with no issues except what I described above.

Also, not sure if this is something that users experience already with other providers or not, as I can only really test with Cogeco.


we need to traverse whole line here... i.e., column variable


This should be using on click btw.


Ai sim


You should use extension to make your code more clear. Example:

```Swift
extension MoviesViewController: UITableViewDelegate, UITableViewDataSource {
    // Implement methods of table view
    func tableView(tableView: UITableView, cellForRowAtIndexPath indexPath: NSIndexPath) -> UITableViewCell {
        // Your code
    }

    func tableView(tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        // Your code
    }
}
extension MoviesViewController: UISearchBarDelegate {
    // Implement methods of search bar
}
```


How did this pass jshint? This should have quotes or use camelCase.


It'd be ideal to generate this lookup table directly from `Buffer.AttributeType` to prevent out-of-sync issues.


Good question. Simply put, chapel.ypp wouldn't compile without it. You will see that each non-terminal of the grammar is listed in a %type line. Which declared the type of the semantic action or $$ for that non-terminal.

Since I added 'required_intent_tag' non-terminal, I am also listing it with the appropriate %type.


then here we can shift i-3, and can start exact match for pattern length, if pattern length matches completely, we return i-pattern.length() (or can add it in array) to return the index of found pattern.


Why is this using isNumeric? If you're just trying to see if show and season are set then just use `if(show && season)` as empty strings, 0 or false will be false.


Can we move this from class scope to file scope now?


I've tried this addition, but I do not see any change in my chart. The bars all turn out the same length.

Any clues to help here?

Thx


not psyched about running this "old school trans" code


Changed in ee5d00678f8930dd40584465d4cc869d1699281b


but you do have a FIXME :)


Fair enough. Back in 2196bd6d2a87799d187cc8cb48f0cbcbd000acef.


Dear Students,

This is the first file we created online.


Good tip.


Is there somewhere these PUA characters are documented?


We can. As the comment above the task notes. For now your change to `restarted` will work. In the near future we can investigate using supervisorctl and/or a role from ansible's galaxy to do the individual restarts. I'll create a separate issue for it...


I want to avoid referencing git in our `package.json`. Perhaps we can publish it to our private repo when that becomes available. I made it a fork because we might want to change what's actually included in the polyfill in the future.


ok (for now)


Why `emplaceBack` instead of `push`?


The `OnClickListener` doesn't need to be set each time the size of the list dips below 1. Instead, it could be set once in `onCreateView` and then left in place, even as the `Button` varies between `VISIBLE` and `GONE`.


The correct solution is to have `trans::collector` produce a list of items to be translated - including closures, instead of recursing the HIR for items and calling `trans_fn`.


Ok, probably then outdated github gem on my end. I just wanted to make sure that the live post is guaranteed to look pretty.

--
Oliver Beckstein
email: orbeckst@gmail.com

Am Mar 15, 2016 um 10:24 schrieb David Dotson <notifications@github.com>:

> I did. I get syntax highlighting locally with the markdown blocks. I think we should keep them.


Oh, so that was why it wasn't switching on the page load for me then


Recursive call is made because of this variable, if you define this in outer scope, then it works.


5521-EMS-InsideHVACSystemIterationLoop-AirLoop (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1024 of 1024 tests passed, 0 test warnings)

![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-30.57%25-green.svg)


true i wasn't sure what a good number was. if the queue can't get to 0 then i think we should decrease again


@OmgImAlexis i didn't run grunt. dont know how. also this "manual_snatch" is the name of the function in webserve.


completely unrelated, but i get villages obstructed by decorations. Im guessing thats an order of generation thing too?


(we have a 15 minute timeout on our own services)


@OmgImAlexis @p0psicles I did "sed" to change somethings. Don't know if changes are OK in "gui/slick/js/core.min.js". need review


A good way of doing this would be to force wget to overwrite the spigot_server.jar. Using cli option -O, you can specify the specific output file and it will overwrite it.
wget -q -O spigot_server.jar https://getspigot.org$URL


Fixes #101


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


Nice!


Может лучше влить в общий дамп )) ??


The way this switches between array and non-array getters makes me nervous. What if we created seperate getters for each component? A `pos` member with two components would be accessible as `pos1` and `pos2`.


Yep, that's what I was thinking too...I always have to double check if it's `-O` or `-o` since curl is the later :)


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#eJzajw8-RbWSf1dG1ij-vg


Have you researched / benchmarked to see if getters and setters have perf overhead vs `get` and `set` methods?


In order to enable shovel plugin the following commands need to be run:
 - rabbitmq-plugins enable rabbitmq_shovel
 - rabbitmq-plugins enable rabbitmq_shovel_management

Also the defined user needs 'policymaker' role in order to create the shovel dynamically
 - rabbitmqctl set_user_tags bo policymaker


TeamCity Emplea.do :: Build - Pull Requests [Build 44](http://137.116.74.213:8111/viewLog.html?buildId=111&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) is now running


TeamCity Emplea.do :: Build - Pull Requests [Build 45](http://137.116.74.213:8111/viewLog.html?buildId=112&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) is now running


TeamCity Emplea.do :: Build - Pull Requests [Build 44](http://137.116.74.213:8111/viewLog.html?buildId=111&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:03


TeamCity Emplea.do :: Build - Pull Requests [Build 46](http://137.116.74.213:8111/viewLog.html?buildId=113&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) is now running


TeamCity Emplea.do :: Build - Pull Requests [Build 45](http://137.116.74.213:8111/viewLog.html?buildId=112&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:03


TeamCity Emplea.do :: Build - Pull Requests [Build 47](http://137.116.74.213:8111/viewLog.html?buildId=114&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) is now running


TeamCity Emplea.do :: Build - Pull Requests [Build 46](http://137.116.74.213:8111/viewLog.html?buildId=113&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:03


TeamCity Emplea.do :: Build - Pull Requests [Build 48](http://137.116.74.213:8111/viewLog.html?buildId=115&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) is now running


TeamCity Emplea.do :: Build - Pull Requests [Build 47](http://137.116.74.213:8111/viewLog.html?buildId=114&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:03


TeamCity Emplea.do :: Build - Pull Requests [Build 48](http://137.116.74.213:8111/viewLog.html?buildId=115&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:03


Can we call this `get` instead of `at`? I prefer verbs for method names.


The changes look good to me.

On Tue, Mar 15, 2016 at 11:31 PM, Mark Fisher <notifications@github.com>
wrote:

> @ilayaperumalg <https://github.com/ilayaperumalg> @jvalkeal
> <https://github.com/jvalkeal> I just added one more commit on this same
> branch: 3e9063f
> <https://github.com/markfisher/spring-cloud-deployer/commit/3e9063f640f5663737cfb461011916cbd54f3ab0>
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly or view it on GitHub
> <https://github.com/markfisher/spring-cloud-deployer/commit/0dad0a8f3126086ddd57d9f3432eb8b6a07d0df0#commitcomment-16714765>
>


> Arbitrary is for generating a random sample of values from some (effectively) infinite set.

I disagree with that statement. I think the question is whether or not we expect ANY `PathCodec` to respect the property or not. I think the answer to that question in this case is debatable, but I would lean towards saying that it is reasonable to expect all path codecs to support lossless serialization.

Sure, `scalacheck` could probably be better designed to support property arguments with non-infinite domains, but I don't think that should affect how we think about properties as that is mostly an "optimization" which, in any case, doesn't really matter all that much if a property has other arguments which do have an infinite domain.


That is weird, I've changed to be inline with other methods.


We ca move the `if (this.length !== this.allocatedLength)` conditional inside `trim`


You can also remove 2 override functions if you don't do any special thing in them.


...and in the description of `-nc`, should be able to have it first check remote availability first:

```
           When running Wget with -N, with or without -r or -p, the decision as to whether or not to download a newer copy of a file depends on the local
           and remote timestamp and size of the file.  -nc may not be specified at the same time as -N.
```


this change really would be helpful to be published to nuget, i literally just encountered this and accessing a private member is why i needed this


I just noticed - this is a method that returns a `boolean` (and follows the convention of ending with a ?), where as the other methods are returning a value. I've changed this back to how it was.


There's email from Roozbeh.  There's also some code in noto-emoji at https://github.com/googlei18n/noto-emoji/blob/master/add_emoji_gsub.py


you could just repush the same commit to a new branch ;)
or close/reopen a PR if there is one open

there is also a restart job button on travis

just saying, cheers !


Can we simplify this signature to always take an object and fork on the existence of an `arrayBuffer` property?


Just wrap it in single quotes then.


Average magnitude of filtered acceleration: `mean(abs(x))`


Sweet


Yes, but we were explicitly clearing it to ensure that any references to the rpmdb were closed.  Are you sure it still works without this?


Le jeu sera apparemment ultra-moddable, donc même dans l'éventualité ou il ne serait pas implémenté (les specs sont pas finales, et tu peux toujours essayer de faire une pull request sur paultech/TC-Specs) ça ne devrait pas être complexe du tout de le faire soi-même.


ok


The test was failing because it could not get sufficient time to detect the "Sign in" after the end of sign out . A fix for this has been performed by adding a waiting time before the "Sign In" is checked.


Where was the PR for this? We need to populate these fields... Can you file an issue for the remaining work?


Hah, I didn't know about `g_set_object`.  Even more C++/smartpointerish.  Okay.


Can we put `StructArrayType` and `StructArray` implementations in separate files?


The current changes cover response serializer and deserializer. In addition, the same changes should be made to request serializer and deserializer.

ServiceRequest class:
https://github.com/petkivim/xrd4j/blob/master/src/common/src/main/java/com/pkrete/xrd4j/common/message/ServiceRequest.java

Serialializer interface:
https://github.com/petkivim/xrd4j/blob/master/src/client/src/main/java/com/pkrete/xrd4j/client/serializer/ServiceRequestSerializer.java

Abstract base class:
https://github.com/petkivim/xrd4j/blob/master/src/client/src/main/java/com/pkrete/xrd4j/client/serializer/AbstractServiceRequestSerializer.java

Deserializer interface:
https://github.com/petkivim/xrd4j/blob/master/src/server/src/main/java/com/pkrete/xrd4j/server/deserializer/CustomRequestDeserializer.java

Abstract base class:
https://github.com/petkivim/xrd4j/blob/master/src/server/src/main/java/com/pkrete/xrd4j/server/deserializer/AbstractCustomRequestDeserializer.java


Rather than duplicating the code to add a crime, you should factor it out. The "extract method" refactoring would make that easy.


Also, you can get back the old behavior with `generate splitSeed`. This means I might as well keep `fastForward`.


I dunno how that top bit is licensed, or I would've edited it too. Might as well leave it alone, to be safe.


If this serializes the *type* it should be a member on `StructArrayType` not `StructArray`


You should be careful with placing too many decorations, like flowers/shrubs/tall grass, so there is enough free space for other mods to add decorations, like botania flowers, etc, though some mods may actually just replace the vanilla decorations with their own, so I'm not sure how big of a deal it would be.


Oh xD
This won't affect performance anyways, it's just another assurance to keep ArrayList.add() O(1)


I think there is something wrong with `self::_strlen($str1) !== self::_strlen($str1)` shouldn't be this `self::_strlen($str1) !== self::_strlen($str2)`?


`createMemberGetterString`


Sorry about the lack of PR. I'll file a work item.


@isidorn reads a bit weird to me, should this not just be a copy of what we do here: https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/parts/output/common/outputServices.ts#L81


Contains the SignIn and SignOut test cases part of the outlook mail automation framework.


In that case, it was to access sub-storage, not SIMD element. Would you like automap to also call the functor on each SIMD element if no overload has been found previously? If yes, then yes I should fix that.


Does this mean that your queue is no longer best option?


`strchr` should be replaced by `strstr` because only an alias!


By selecting between the two options here in `onCreateView`, you've frozen it in place even though crimes can be added and deleted, which could transition the list from empty to nonempty or vice versa. To make the interface responsive to these changes, you should move this code to `updateUI`, which has the responsibility for bringing the user interface into conformance with the current state of the list.


It's the second time in two weeks that both of us get caught by this.

 I now think it would be nice, to express many chess ideas, to have an array  `rectangle_bb[square, square]` (along with its accessor function) which would contain the bitboard of a rectangular area on the board.

For instance, `rectangle_bb(C2, F4)' would be the squares C2,D2,E2,F2,C3,D3,E3,F3,C4,D4,E4,F4.


ok. (sigh. we should really patch this up.)


I find this diff pretty straight forward, I'm happy to amend the commit note if you have a suggestion that might make it clearer?


I'm not clear what you've gained by wrapping a `RelativeLayout` around the `Button`.


ok


Seems to work alright here. Although that whole function was gutted and moved to SysrootUpgrader (but the same remains true, i.e. I don't explicitly clear the hifctx at the end).


Suggest changing to ``tahoe``.


s/python/Python/


I saw major perf gains by shortening the _source_ of these generated methods (i.e. renaming `elementIndex` to `e`) (likely due to v8 inlining heuristics).


@arielb1 has opened rust-lang/rust#32202 which we can land after this PR.


I just had to make it phonetically accurate.


Put the command on a separate line.


s/python2.7/Python 2.7/


Just curious... why is this needed when it never gets called without [checking first](https://github.com/reactioncommerce/redoc/blob/36aa86fa8234a4e0dcce2da7480ec6064fb96b4f/common/main.jsx#L18)?


Put this on a separate line.


Otherwise LGTM.


It does not follow PSR-4


I generally advise using the most general type that is consistent with how a variable will be used; that preserves as much flexibility for later changes as possible. In this case, if you don't use any of the features that are specific to `LinearLayout`, you might consider using the more general type `View`. That way the XML file can be changed to a different kind of layout and the Java code wouldn't need to be altered correspondingly.


@flavour Please review


Note there is  `glnx_mkdtempat()` now too which already takes a `mode`.


@flavour Because ```'set' object does not support item assignment``` This is the error I received when I tried settings.L10n.languages["my"] = "မြန်မာစာ"


That would make more sense, I've raised a Trello card to look at updating that.


@Tomaco , it seems to be typo `before_filer`  :eyeglasses:


sdsg


The code for adding a crime is the same in two places. Rather than duplicating it, you should factor it out. An "extract method" refactoring would make that easy.

Also, there's no need to re-establish the `OnClickListener` each time the list transitions to being empty. You could set the listener once in `onCreateView` and just leave it in place. During periods when the list is nonempty, the `Button` won't be visible but the listener won't be doing any harm, and when the list becomes empty and the `Button` is revealed, the listener will be ready and waiting.


I'd prefer to write our code using arrows anyway (can't do without anymore), so node 5 support is ok for me. I think we can always add compilation in a later version.


omg so much better


Immutability is a good property for classes. Especially for data-like classes which tend to be stored in some collections. Using mutable classes can break some contracts the places they're stored because of external changes in leaked references.

Don't use public && not final fields.


> 3\. For each layout box in the viewport, in paint order, starting with the topmost box, that would be a target for hit testing at coordinates x,y even if nothing would be overlapping it, when applying the transforms that apply to the descendants of the viewport, append the associated element to sequence.

Does this means I don't have to do hit_test anymore because `mouse_over` / `nodes_from_point` automatically calls `hit_test`?

Because of parameters & returns:

    fn mouse_over(point: Point2D<f32>) -> Vec<UntrustedNodeAddress>
    fn hit_test_query(&self, hit_test_request: Point2D<f32>, update_cursor: bool) > Option<UntrustedNodeAddress>
    fn hit_test(&self) -> HitTestResponse

I'm not sure if I can map the output of mouse_over to `hit_test_query`/`hit_test`


Completeness: 5
Comfort: 3-4

I'll keep reviewing the CLI commands until they become habit.


Yeah, but then I'd have to open an fd on the parent dir. :)

Oh, or could I do `glnx_mkdtempat (FD_ATCWD, "/absolute/path/dir-XXXXXX", ...)` ?


Maybe then just repeat the descriptions, "The active and inactive EFI partitions are necessary..."


and node 5 supports deconstructing. so let's go with that
On Mar 15, 2016 19:28, "Jasper Woudenberg" <notifications@github.com> wrote:

> I'd prefer to write our code using arrows anyway (can't do without
> anymore), so node 5 support is ok for me. I think we can always add
> compilation in a later version.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub
> <https://github.com/stoeffel/awesome-fp-js/commit/b9f0abcb62c39f8f87d0127143ebe306b1b92db9#commitcomment-16715331>
>


nice


ini yang kurubah bro..


It's from the old `mouse_over` code. Recently I've found the `update_cursor: false` parameter. Will look into it, thanks!


Yeah - was wrong. It closes and already open demuxer for a possible other format -> all fine. Was early that morning, no negation needed.


Are you sure that -1 isn't a special value un GLua?


not sure i understand this error case?


lol, no worries. I do it all the time. Sorry if I came off a bit brusque. I ain't mad, Bro. Thanks for filing.


this was just a bug before I guess? (the case of `nested_returns && mir.is_some()`)


LGTM


Do we have this domain?


I think actually the whole comment block can be removed. It seems to be left over from when the plan was to call enterLameduck() from runHealthCheck(), but I later decided to call it from changeCallback() instead. So it doesn't make sense to explain why that one code path in runHealthCheck() lacks enterLameduck(), since no code paths in runHealthCheck() are expected to have enterLameduck() anyway.


And PopScope doesn't do any extra cleanup that survives LoadScopes?


ugh I hate this part about how we currently represent patterns


Good catch, I've added removing those to the commit.


`capitalize` is also a valid [text-transform value](https://developer.mozilla.org/en-US/docs/Web/CSS/text-transform) :-). Nbd though `uppercase` is fine too (capitalize only targets the first letter, whereas uppercase capitalizes the whole string).


One can use `this(random.nextInt(), ...)`


The code previous to this change was shorthand for doc ready


`template` looks like it's always absolute in the callers, so indeed we'd just need to use `AT_FDCWD` it looks like to me.


k


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 140](https://teamcity.labkey.org/viewLog.html?buildId=310613&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:02:38


Reading through this again, this sentence would ready better as, "A simple customization would be to adjust the size of the partitions to better match the space available on a specific media device."


You can use $(function(){}); for executing on DOM load


um... tried to use octocat to deploy to github but he refused. Yet he says he did something.


I've added the following to the commit note:
```
There are no changes in the test artefacts by this PR,
confirmed by running the regression tests producing no
diffs.
```


This doesn't work on Windows 7 64 bit latest sublime 3103 for me. It's showing blackbox when it exceeds 100 line. Could you look into this? Thanks


ok


Yeah. Ideally, MIR would not use the old trans return logic, but for now there's no point duplicating it.


thank you


jQuery makese these a little easier to write and is backed by the same document.querySelector method for performance (when it is advantageous):
    $('#toast-notification');


TeamCity FubuMVC Swank :: Deploy [Build 2.0.64.0](http://build.mikeobrien.net/viewLog.html?buildId=2542&buildTypeId=fubumvcswank&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 399 Build time: 00:01:40


Shouldn't have been included to begin with :+1:


In which file is that hard-coded for courses too? I'm pretty sure courses are given a different type.

If you're saying that `http://lrs.learninglocker.net/define/type/moodle/` returns a 404 that's acceptable in the spec.


@wwwmhcom estou usando seu Gerador de assinaturas como base e tenho que deixar uma opção para a pessoa escolher a empresa que trabalha dependendo da empresa ela troca a imagem da assinatura vc saberia como possa add está função no seu gerador tentei fazer isso:

`<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">

<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<meta charset="UTF-8">
<title>:: ASSINATURA ::</title>
<link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.1/css/bootstrap.min.css">
<link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.1/css/bootstrap-theme.min.css">

                        <form action="" method="post" onSubmit="return valida_dados(this)">

                                  <input type="text" class="form-control" placeholder="Nome" name="nome" />

                                  <input type="text" class="form-control" placeholder="Departamento" name="depto" />

                                  <input type="tel" class="form-control" placeholder="Telefone - apenas números com DDD" name="tel" onkeypress="mascaraTelefone(this,'## ####-####')"  maxlength="14"/>

                                  <input type="tel" class="form-control" placeholder="Celular" name="cel" onkeypress="mascara(this,'## #####-####')" maxlength="13"/>

                                  <input type="text" class="form-control" placeholder="Local" name="local" />

                              <input type="radio" name="escola" class="form-control col-xs-offset-0 col-xs-5" src="microcamp.png" />

                              <input type="radio" src="mc_tech.png" name="escola" class="form-control col-xs-offset-0 col-xs-10" />

                              <input type="radio" src="mc.png" name="escola" class="form-control col-xs-offset-0 col-xs-10" />

                              <input type="radio" src="grupo_mc.png" name="escola" class="form-control col-xs-offset-0 col-xs-10" />

                                <button type="submit" class="btn btn-primary" name="Enviar">Gerar Assinatura</button>

                              <?php
if(isset($_POST['Enviar'])){
$nome = $_POST['nome'];
$depto = $_POST['depto'];
$tel = $_POST['tel'];
$cel = $_POST['cel'];
$local = $_POST['local'];
$escola = $_POST['escola'];
echo "

     $escola
    $nome | $depto
$tel / $cel
$local

    Desde 1977 ensinando.

";
}
?>

                        </form>

por: Marcos Henrique

<map name="Map" id="Map">
  <area shape="rect" coords="15,154,102,189" href="suporte.php" alt="MPLS"/>
<area shape="poly" coords="149,160" href="#" /><area shape="poly" coords="147,165" href="#" />
<area shape="rect" coords="1068,142,1184,178" href="mailto:marcos@marcoshenrique.com" alt="MAP" />
<area shape="rect" coords="-1,-9,1206,136" href="index.php" />
</map>

`
Mas não está retornando a imagem :(


my bad. this has to be SpilledRegisterArg


This layout is almost the same as default pagination, I don't think we need to create a copy-paste here.

I see the difference that default _pagination template has:
  {{range}} of {{count}}

Which is wrong to have in hbs template anyways since we need to i18n it, please unify these templates.


5521-EMS-InsideHVACSystemIterationLoop-AirLoop (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Možná by bylo lepší to tam nemít zakomentované, ale implementovat to jako 8bit metodu čtení.. Třeba něco jako get_hub_8bit().


This is not your folder.


In hindsight, I probably should have done import/export stuff there anyway (instead of relying on an `analytics` global).


TaskCluster: @njirap does not have permission to trigger tasks.


We shouldn't create calculation conditions basing on template name, would the checking of count be the sign that we are in infinite mode?


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 137](https://teamcity.labkey.org/viewLog.html?buildId=310615&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:05:06


CardsListContent.items is used only in this place hence there is no need to open this data. It could be replaced with `CardListContent#getIdAtPosition(position)`

Not mentioning NPE and AIOOBE in this line.


If we combine pagination template, this table is a gonner.


Traditionally, script references go in the head, is there a reason why you want them in the body?


ok


I'm also noticing something about getspigot's outtage right now... If you access the same files over http, it works fine. Something must be broken with their SSL on the backend behind cloudflare. Maybe we could also add a way for the script to fallback to http when https fails?
I'm experimenting with some code right now and i'll comment my findings.


we also discover that and removed `dev` from our sdks.


I'm not 100% sure how this code works -- but can we just ignore `args` altogether and always use `llargs[2]`?


Cool, sounds good. Will refactor it and drop this commit then.


heh, yes


Feel free to submit a pull request if your experiments go well.


@ichord This same check is happening at a [later point](https://github.com/ichord/At.js/blob/e24b563e4d85681d683b8c1f7e345c98f2da3cb2/dist/js/jquery.atwho.js#L832#L834) in the same function.


SImple change,


ah nice


Count will come through whether it's infinite or not, the only difference being count displays how many items has been returned. I'll change the method of flagging.


Yeah, will try to explain on slack.


I'll change the strategy appropriately.


I believe that my first suspicion was wrong; the patch seem to be correct (after you've explained the details).
So - though I'm not proud to admit I was wrong - I must apologize for the noise. ;)
.. I can't promise I won't make noise in the future, though. :D


Done.


Ah, right... my bad. Somehow I missed the visual change when trying it out :p


No, because this code *doesn't really work* atm, as MIR is still missing rvalue promotion, I was just trying to get `"shuffle index #{} is not a constant"` below to show up.
This is where being forced to pass a constant value would come in handy for solving the problem at the type level.


I agree that all PathCodecs should pass these tests, and probably some others as well. The situation is somewhat analogous to when we do, for example, `checkAll(traverse.laws[LogicalPlan])`. We do a similar thing when we want to verify streaming vs. non-streaming query evaluation, or PUT vs. POST. I just think it makes sense to think about covering these spaces in a different way than what Arbitrary does for you.

I'm sure there's a clearer way to say this, or maybe somebody else on the team has another way of looking at it. Maybe we should bring them in? (realizing now that these comments on the commit aren't getting mirrored in Slack).


Yeah, in this case maybe we need to introduce the paginated-view property, something like infinitePagination: true or false


yes sure, but how do we load classes which don't follow it ? I prefer PSR-4 classes, but all external projects don't do it...
Removing this implies we must use another loader when mixing Phalcon and external classes which is in my opinion the contrary of using a framework.

Saying that you don't recommend using it for performance would be good, but removing it is a bit harsh...


Done


## Task 1

Work your way through the entire card/deck example of **inheritance** that is given in [Chapter 18](http://www.greenteapress.com/thinkpython2/html/thinkpython2019.html) of our book. By the time you finish, you should have created a file containing three classes (`Deck()` and `Card()` and `Hand()`).  Be sure to include all the attributes/methods covered in the example.  Also be sure to include the `Deck.sort()` method that is requested.  You can use a `main` function to demonstrate (and show yourself) how these classes work independently and together.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program includes the requested attributes and methods (4.0 pt)
- [ ] Program demonstrates it's functionality (e.g. show it working from `main()` or similar) (2.0 pt)

## Task 2

Rethink your word-counting code just a little bit.  How could you parse and summarize words in a text file using an object-oriented approach?  Write a class that operates generically, on any file, to summarize word counts.  Specifically, write a class that includes or computes the following attributes:

* file name (this is the name of the actual file)
* file path (this is the path to the file)
* total number of words in file
* type of puncutation used in the file

And also that includes the following methods (these may not be all you need, just the ones you must have):

* a method to count words
* a method to count punctuation
* a method to display both of these results

Your program should contain a main loop and an ifmain statement, it should be formatted correctly, and you should use argparse to get the name of the input file from the user.  Your program should display, to the user, (1) the top 20 words and their counts, and (2) the top 10 puncation marks and their counts.  You do not need to write any results to an outfile.  Be sure to include the file(s) that you used to test/design your program.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse` to get the input filename (1.0 pt)
- [ ] Program uses a class-based approach and includes the requested attributes and methods (4.0 pt)
- [ ] Program correctly counts and displays both words and puncutation in any file (2.0 pt)


fixed

On 15-03-16 17:13, Bart van der Zwet wrote:
>
> No javadocs is better than empty javadocs.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub:
> https://github.com/OSGP/Shared/commit/1088d356822f401e33d04d559ad1fe502a053e12#commitcomment-16711875
>


Yes that part is a bit of a pain really. I think we're reaching the limit of configuration management tools in general with these kind of operations. Probably best left to (manual) orchestration indeed.

And then there is the search head deployer that overrules any unmanaged entries in etc/apps on the SHC nodes, like the config apps this Puppet modules now generates. So config apps for an SHC node actually need to be deployed in etc/shcluster/apps on the SHD. Headache!

Installation of some apps on a Splunk SHC already requires a staging instance, so perhaps it's best to follow this route too. [Source](http://docs.splunk.com/Documentation/ES/latest/Install/InstallEnterpriseSecurity).


what does this fix ? someone knows the fix for the android.process.phone fc ?


TaskCluster: @njirap does not have permission to trigger tasks.


sempre più avanti


again, mutability.

moreover, storing such data in static variables is something strange. Semantics of the class — "Content" says that instance of that class is some content. In your case instance of this class stores nothing.

Also, http://stackoverflow.com/questions/19774075/is-it-bad-practice-to-keep-data-in-static-variables


Either way this check is temporary, so no reason to ponder over it too much?


This class is used nowhere explicitly so can be private


public, not final


Nice


não deveria ser:
`if  ((negativo) && (res != 0)){ `


Interesting !

For now, we have only SpaceMask and OutpostRank that uses many ranks or files,
and they are used only as const.
So maybe not worth the trouble...

Better to (always) double check our code with some visual printouts such as

`sync_cout << Bitboards::pretty(b) << sync_endln;`

or

`sync_cout << pos << sync_endln;`


Thanks for that , I am not sure how the github works kinda hard to navigate

Sent from my iPhone

> On Mar 15, 2016, at 10:30 AM, Gene Leynes <notifications@github.com> wrote:
>
> As the project owner, can't you delete any comment, or just yours?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub
>


5521-EMS-InsideHVACSystemIterationLoop-AirLoop (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1582 of 1582 tests passed, 0 test warnings)

![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.86%25-green.svg)


There are warnings in Idea saying you're using STRING in schema instead of TEXT.


look at the panel watch screen it handles multiple temperature controls.


Qual foi o cheat pra chegar nessa lógica?


can be static, does not use the object state.


You know StyleCI can do this for you. ;)


Steve,
  If you do not mind, I am going to ask a series of questions related to the Coffeescript/jQuery/AJAX techniques used for editing the To Do List Item.  I am thinking about making some enhancements before Saturday, so before I start making changes I really need to understand how this flow works.  The changes might be simple but I really have taken an interest in this front end coding and how it interfaces with Rails.


Infact, if you enable auto sending and merging, it'll do it without you even needing to ask it to.


Can we please move these action types to `client/state/action-types.js` ?


If I remember it correctly XPath with attributes are not supported in Python 2.6. As a result, the test case passes with 2.7 but fails with 2.6:
```
$ youtube-dl python2.7 -vF test:noz
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-vF', u'test:noz']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2016.03.14
[debug] Git HEAD: 3ff8279
[debug] Python version 2.7.11 - Linux-4.4.5-1-ARCH-x86_64-with-glibc2.2.5
[debug] exe versions: avconv v12_dev0-2434-gf6ccee9, avprobe v12_dev0-2434-gf6ccee9, ffmpeg 3.0, ffprobe 3.0, rtmpdump 2.4
[debug] Proxy map: {}
[TestURL] Test URL: http://www.noz.de/video/25151/32-Deutschland-gewinnt-Badminton-Lnderspiel-in-Melle
[Noz] 25151: Downloading webpage
[Noz] meta configuration: Downloading webpage
[Noz] video configuration: Downloading XML
[Noz] 25151: Downloading f4m manifest
[Noz] 25151: Downloading m3u8 information
[info] Available formats for 25151:
format code  extension  resolution note
hls-96       mp4        audio only   96k , mp4a.40.2
hls-meta     mp4        multiple   Quality selection URL
http-lq      mp4        400x225     250k
hds-256      flv        unknown     256k
hls-256      mp4        192x108     256k , avc1.64001f,  mp4a.40.2
http-mq      mp4        512x288     550k
hds-595      flv        unknown     595k
hls-595      mp4        512x288     595k , avc1.64001f,  mp4a.40.2
http-hq      mp4        512x288     750k
hds-825      flv        unknown     825k
hls-825      mp4        640x360     825k , avc1.64001f,  mp4a.40.2
http-hp      mp4        512x288    1200k
hds-1651     flv        unknown    1651k
hls-1651     mp4        1024x576   1651k , avc1.64001f,  mp4a.40.2
hds-1958     flv        unknown    1958k
hls-1958     mp4        1280x720   1958k , avc1.64001f,  mp4a.40.2
http-hd      mp4        512x288    2000k  (best)
```

```
$ youtube-dl python2.6 -vF test:noz
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-vF', u'test:noz']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2016.03.14
[debug] Git HEAD: 3ff8279
[debug] Python version 2.6.9 - Linux-4.4.5-1-ARCH-x86_64-with-glibc2.2.5
[debug] exe versions: avconv v12_dev0-2434-gf6ccee9, avprobe v12_dev0-2434-gf6ccee9, ffmpeg 3.0, ffprobe 3.0, rtmpdump 2.4
[debug] Proxy map: {}
[TestURL] Test URL: http://www.noz.de/video/25151/32-Deutschland-gewinnt-Badminton-Lnderspiel-in-Melle
[Noz] 25151: Downloading webpage
[Noz] meta configuration: Downloading webpage
[Noz] video configuration: Downloading XML
ERROR: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "<string>", line 4, in <module>
ImportError: No module named importlib
Traceback (most recent call last):
  File "/home/yen/Executables/Multimedia/youtube-dl/youtube_dl/YoutubeDL.py", line 668, in extract_info
    ie_result = ie.extract(url)
  File "/home/yen/Executables/Multimedia/youtube-dl/youtube_dl/extractor/common.py", line 320, in extract
    return self._real_extract(url)
  File "/home/yen/Executables/Multimedia/youtube-dl/youtube_dl/extractor/noz.py", line 73, in _real_extract
    self._sort_formats(formats)
  File "/home/yen/Executables/Multimedia/youtube-dl/youtube_dl/extractor/common.py", line 839, in _sort_formats
    raise ExtractorError('No video formats found')
ExtractorError: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
```
From the git log this problem exists before this commit. Though I guess it's better to fix it as well this time.


sock_cntr_inc always returns 0. Made it void.


I think in vanilla villages are placed early, because I've seen trees and ponds take chunks out. Are we doing villages later, after other decorations have run? I see enough villages - maybe it's a 1.8.9/1.7.10 thing?

It's really only Flower Field that's totally over the top with decorations, so not really a global problem. Having seen it, the total riot of color is the fun (and it is fun). That said, some variation would be nice because the flowers-everywhere covers so much space it deadens the effect (no contrast), although that wouldn't be true to the source, which is also totally plastered with flowers.


Added assert.


test


How about nothing...


To be consistent and avoid premature optimizations we should trim all nested objects above some threshold (more than 2 fields) and all nested lists from responses on collection resources (xxx_GetList operations). For responses fetched on singleton resources (xxx_Get operations) we should have everything included by default.


We need to localize 'of', so I guess we need to calculate the whole text string in the code. please update. I belive we already have a localization resource like "%page of %count"


gostei dessa reescrita


Update Morphs topic to use '::' and find deep submorphs; owningWindow


indeed.


Do not forget to make an empty line at the end of every file.


damit im device auch der Kanal enthalten ist, sind noch einige Anpassungen nötig:
Für den $deviceCode fallen mir 2 Varianten ein:
```
my $deviceCode = $id . '_' . $BitChannel;
oder
my $deviceCode = $id . '_' . $Channel;
```
in den folgenden Zeilen in der Parse Routine müssen noch die $id durch $deviceCode ersetzt werden
```
my $def = $modules{Dooya}{defptr}{$deviceCode};
..
	} else {
		Log3 $hash, 3, "Dooya Unknown device $deviceCode, please define it";
		return "UNDEFINED Dooya_$deviceCode Dooya deviceCode";
	}
```

In der sub sendcommand muß dann der devicecode durch split wieder in $id und $channel aufgeteilt werden


You're welcome ;) I however recommend considering [Qsmtp](https://github.com/DerDakon/Qsmtp) instead, for that I have even automated tests that can check such cornercases ;)


Consider to use SF's `@AliasFor`.
See an example in the `@ComponentScan`.


saquei, mas vc poderia usar a abordagem inversa, ao invés de decrementar o valor, vc começava somando os ímpares até alcançar `num1`, a quantidade de iterações na soma seria a raiz


Oops... I'll reschedule with correct bounds...


:smile:


MDN recommends using `<defs>` (ref: [here](https://developer.mozilla.org/en-US/docs/Web/SVG/Element/defs)):

> Defining these elements inside of a defs element promotes understandability of the SVG content and thus promotes accessibility

Though using `#legend` under `<svg class="main-svg">` could lead to conflict on DOM queries. I'll throw in a `uid` in the id to be safe.


Well, really looks like you should rely on the `AbstractReplyProducingMessageHandler` and don't worry about the `messagingTemplate`.
Pay attention to the `requiresReply = false`


I share this concern. I struggled with this when writing a user-defined reduction.

To give some details, currently the variable marked with a reduce intent has, in the body of the loop, the type given by ReduceOp.identity. So it does not necessarily support accumulate() method. We have considered providing reduce= operator, we are not settled on it and do not provide it at this point.

We definitely need to and will work out how reduce-intent variables are to be referenced in the loop - and outside the loop, too, for that matter. Doing so is beyond the scope of this PR. So I suggest either accepting this example as-is or removing the user-defined reduction from it.


já te adicionei!


I Used that at first. But there was A reason. Have to check what that was.


Please merge this?


Done. I was following the pattern in other tests.


## Task 1

Work your way through the entire card/deck example of **inheritance** that is given in [Chapter 18](http://www.greenteapress.com/thinkpython2/html/thinkpython2019.html) of our book. By the time you finish, you should have created a file containing three classes (`Deck()` and `Card()` and `Hand()`).  Be sure to include all the attributes/methods covered in the example.  Also be sure to include the `Deck.sort()` method that is requested.  You can use a `main` function to demonstrate (and show yourself) how these classes work independently and together.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program includes the requested attributes and methods (4.0 pt)
- [ ] Program demonstrates it's functionality (e.g. show it working from `main()` or similar) (2.0 pt)

## Task 2

Rethink your word-counting code just a little bit.  How could you parse and summarize words in a text file using an object-oriented approach?  Write a class that operates generically, on any file, to summarize word counts.  Specifically, write a class that includes or computes the following attributes:

* file name (this is the name of the actual file)
* file path (this is the path to the file)
* total number of words in file
* type of puncutation used in the file

And also that includes the following methods (these may not be all you need, just the ones you must have):

* a method to count words
* a method to count punctuation
* a method to display both of these results

Your program should contain a main loop and an ifmain statement, it should be formatted correctly, and you should use argparse to get the name of the input file from the user.  Your program should display, to the user, (1) the top 20 words and their counts, and (2) the top 10 puncation marks and their counts.  You do not need to write any results to an outfile.  Be sure to include the file(s) that you used to test/design your program.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse` to get the input filename (1.0 pt)
- [ ] Program uses a class-based approach and includes the requested attributes and methods (4.0 pt)
- [ ] Program correctly counts and displays both words and puncutation in any file (2.0 pt)


Yeah omg your right. Was copy/past from legacy.


I assumed that there is no need to "upgrade" `buildbot-slave` to `buildbot-worker`.

If user want to use existing `buildbot-slave` in nine he will just use it, since it will work with nine master.
If user deploys new workers, he should just use new `buildbot-worker`.

The only use case that will require "upgrade" `buildbot-slave` to `buildbot-worker` is if user would like to use new features of `buildbot-worker` that are not yet expected.


Would be better that `BPP`s are registered as `static`.


getting db may result in exception. One can return emptyList() in that case, for example


Thanks, makes sense!


Ahh. Do the `defs` need to be top level or somewhere else? Seems like a bit of a pain having to position two elements absolutely rather than just letting a containing `<g>` handle it and do relative positions.


I'll go through it and run grunt.


5521-EMS-InsideHVACSystemIterationLoop-AirLoop (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (2132 of 2132 tests passed, 1 test warnings)

 * 1 test had: EIO diffs.
 * 1 test had: ESO small diffs.
 * 1 test had: MTR small diffs.
 * 1 test had: Table small diffs.

![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)


Shouldn't we rely on the `messageBuilderFactory` from the `BF`?


5521-EMS-InsideHVACSystemIterationLoop-AirLoop (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2170 of 2170 tests passed, 1 test warnings)

 * 1 test had: EIO diffs.
 * 1 test had: ESO small diffs.
 * 1 test had: MTR small diffs.
 * 1 test had: Table small diffs.

![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)


OK, agreed. Thanks!


```
try:
    get_status_code()
except RequestException:
    self.err("Unstable connection.")
```


What is the reason of this extra `Map`, if we can just use `MessageBuilder.setHeader()` ?


this is way better than using this `staticmethod`-`self` weird construct^^


gave up the PR, there were some wrong replaces. too lazy to re-do


NPE


5521-EMS-InsideHVACSystemIterationLoop-AirLoop (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


Maybe - I think this part needs to be somewhat refactored. Not sure that the converters in general should return Message objects ...


Fun times XD , good luck to all


the opening " is missing here


5521-EMS-InsideHVACSystemIterationLoop-AirLoop (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1024 of 1024 tests passed, 0 test warnings)

![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-30.57%25-green.svg)


Otherwise it is a good idea.
Some `@ServiceActivator` reborn :smile:


Never used, should be removed.


Do you really need dbHelper and smsProvider for each fragment? Or they could be passed in constructor?


Did I forget to push the fix? Huh.


Try to separate this line to something like:
snprintf(msg, 128,
  "Found %016lux @ %i  #%i  (table:%i)\n"
  "Kraken>",
  result, mBitPos, mJobNum, mAdvance);

It is better to read separated short lines than one long.


Did you mean for antler_caress to be a form of attack? I'm a bit confused, since the prototype for antler_caressed prints the string (botWhoCaresedMe.name + "attacked me!"), but to caress someone is to  touch or stroke gently or lovingly


Thanks for reminding me. I spent some time understanding the implementation of the instantiation service, but then I didn't use it.


Какого черта ты передаешь сюда 0, а не false?


Are we using live rubygems intentionally here? I noticed this test failing or being slow.

I tried to make this a fake invalid gem with build_gem and failed :smile:.


system("pause"); лучше


Add a new reflection


Hey guys.  The `noscroll` setting defaulting to `true` hoses Chrome desktop, tablet, and phone scrolling.  What do I mean by hose?  It means that if I have a "long" sign-in modal the user is unable to scroll down to the "sign in" button - its chopped off.

I guess I can go through our code base and data-options="no_scroll: false".

Or I can submit a pull request to default to false?  What do you think?  Thoughts?
![capture](https://cloud.githubusercontent.com/assets/156634/13790547/6d08f01c-eabf-11e5-8580-19925632ed3e.PNG)


Fix dummy white spaces/tabs.


*sémantique ptn


Давай переменным нормальные имена, а то как быдлокодер какой-то:
`Mover mover; // например`


## Task 1

There are 2 chapters of Darwin's Origin of Species in this repository within a directory entitled `task1-files`.  Write a program that uses `argparse` and `glob` to open and read these chapters and some other code to get a list of all words in each chapter.  Then output, to the command line, a pretty-printed list of the following:

* the total count of words in Chapter 1
* the count of unique words in Chapter 1
* the total count of words in Chapter 2
* the count of unique words in Chapter 2
* the count of words in Chapter 1 that ARE in Chapter 2
* the count of words in Chapter 1 that ARE NOT in Chapter 2
* the count of words in Chapter 2 that ARE NOT in Chapter 1

Your program should contain a main loop and an ifmain statement, it should be formatted correctly, and you should be sure to use argparse to `argparse` and `glob` to read in the chapter files.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse` and `glob` (1.0 pt)
- [ ] Program functions as requested (3.0 pt)

## Task 2

In the directory `task2-files`, there are a lot of files.  Write a program that uses `argparse`, `glob`, and `os` to copy **only** the fastq files to a new directory.  Your program should contain a main loop and an ifmain statement, it should be formatted correctly, and you should take both the input directory containing the files you want to copy and the output directory where you will copy the files as arguments on the command-line.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse`, `glob`, and `os` (2.0 pt)
- [ ] Program functions as requested (2.0 pt)

## Task 3

In the directory `task3-files` there are 10 files, each containing a random selection of integer values.  Using a combination of `argparse` and `glob`, read these files.  Then, use `map` to compute the sum of all integer values across all files. Your program should contain a main loop and an ifmain statement, and it should be formatted correctly.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse`, `glob` (0.5 pt)
- [ ] Program uses `map` (2.0 pt)
- [ ] Program functions as requested (1.5 pt)


@joaquimrocha Makes sense, I forgot that keep-above does not work for maximized windows.


Hi @jypuigbo and @dcam0050,

I created this script to easily update the whole software related to yarp/icub/wysiwyd at once. I suggest you adapt this on your machines. Please let me know if you have any questions :).

Best,
Tobi

/cc @AnneLaureM @gregoire-pointeau:
This script is installed on all of your machines! Please use it :)


whaaaat noooooo live rubygems specs have to be tagged "realworld" and live in a different place. this seems really bad :grimacing:


@pattacini: I am not sure whether this would be useful for your setups at IIT.


`HANDLE handler = ...` – объявил и сразу же присвоил значение, чего размазывать по коду-то?


function replaced


good catch! fixed.


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


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


Not localisation-friendly


findData двинь как можно ближе к первому его использованию. То есть прямо перед вызовом FindFirstFile:

```
WIN32_FIND_DATA findData = {};
HANDLE handle = FindFirstFile(mSourceStringBuffer.c_str(), &findData);
```


This is unused. WTF?


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


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


I agree with you.

I thought of the solution you pointed out earlier about `getObtainedCallStack`. However I didn't want to complicate the decision of someone trying to get the call stack, and not knowing which method to call.

I'll think about it and update this comment.


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


right, i chose to avoid redundancy instead of an explicit flag indicating pixel data.

Better might be a comment for the codec field above:

```
AVCodecID codec;  // The codec, or AV_CODEC_ID_NONE for pixel data or unknown

AVPixelFormat pixfmt; // Used for pixel data
```


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


Thanks. Do you happen to have a list of patterns you use? I try to follow the code conventions and patterns, but some (like this) are hard to spot!


5521-EMS-InsideHVACSystemIterationLoop-AirLoop (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (2137 of 2138 tests passed, 4 test warnings)

 * 4 tests had: EIO diffs.
 * 4 tests had: ESO small diffs.
 * 4 tests had: Table small diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: Table big diffs.
 * 1 test had: MTR small diffs.

![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)


yeah, no fix on the 7.2.x branch pushed


Andrew, you are a god


Sorry, I do not use Emacs myself and was not aware it was required in order to use the snippets. Please ignore ;)


Its d1ef none of the nethunter hack you added.. same reboot behavior i have been having with pure stock..


Why did you restore the redundant "Mirrors" section which I removed in https://github.com/QubesOS/qubesos.github.io/commit/1064a349debc298750c2dfe16a45567ba884b505?

Do you not agree that it is redundant, given that we already have literally 18 links to `mirrors.kernel.org` on that page?


Not a very good name


Clean up Maltego notes.


[Käyttöliittymän ja tietokannan suunnitelma.pptx](https://github.com/k6058/IIO11300/files/174630/Kayttoliittyman.ja.tietokannan.suunnitelma.pptx)
[Sovelluksen toiminnot käyttötapauskaavio.pptx](https://github.com/k6058/IIO11300/files/174631/Sovelluksen.toiminnot.kayttotapauskaavio.pptx)


Not needed parameter


1.39.0 available
And stop using the :0:4 thing


All looks to be working fine in 0315. :+1:


Harjoitustyön suunnitteludokumentteja lisätty


@osm0sis Thanks for the feedback!


This could possibly be local static method here.
And for sure should not store state in some strange place


kk


Agreed, that's what I meant. Making it configurable or just make invoices part of the permission list.


@Override


Ajax with Asp.net Mvc project This has diffent features of Ajax in Asp.net Mvc OnBegin, OnSuccess, gridrow count


Is 1.39.0 a testing version though?


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


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


why public


Par chybicek opravim zitra :D greyed a write to us je az moc czechenglish :(. :D


Sure. I might not get back to you right away, but I'll definitely answer what I can.


Yep, after reading some code it's totally unclear how and when data there should be initialized. Very strange data retrieval.


cc @luislobo :+1:


unused


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


Script always goes at the end of the body so that it has access to all the Dom objects, because when u moved it to the top, the progress bar wasn't being initialized properly since the object wasn't available when the script ran before everything else.

On March 15, 2016 1:36:21 PM CDT, Charlie Hayes <notifications@github.com> wrote:
>Traditionally, script references go in the head, is there a reason why
>you want them in the body?
>
>---
>You are receiving this because you authored the thread.
>Reply to this email directly or view it on GitHub:
>https://github.com/idontusenumbers/JobMatch/commit/b2c316cba29d4f912fcd7d75c0ec9f645bbfef96#commitcomment-16715516

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.


This `get_val()` method is probably unnecessary.  `some_node.val` will return the same thing.


Had to revert to this as trigger_section and user_group were not saving in DB.


There is some warning about date localisation, I'd respect that


## Task 1

Work your way through the entire card/deck example of **inheritance** that is given in [Chapter 18](http://www.greenteapress.com/thinkpython2/html/thinkpython2019.html) of our book. By the time you finish, you should have created a file containing three classes (`Deck()` and `Card()` and `Hand()`).  Be sure to include all the attributes/methods covered in the example.  Also be sure to include the `Deck.sort()` method that is requested.  You can use a `main` function to demonstrate (and show yourself) how these classes work independently and together.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program includes the requested attributes and methods (4.0 pt)
- [ ] Program demonstrates it's functionality (e.g. show it working from `main()` or similar) (2.0 pt)

## Task 2

Rethink your word-counting code just a little bit.  How could you parse and summarize words in a text file using an object-oriented approach?  Write a class that operates generically, on any file, to summarize word counts.  Specifically, write a class that includes or computes the following attributes:

* file name (this is the name of the actual file)
* file path (this is the path to the file)
* total number of words in file
* type of puncutation used in the file

And also that includes the following methods (these may not be all you need, just the ones you must have):

* a method to count words
* a method to count punctuation
* a method to display both of these results

Your program should contain a main loop and an ifmain statement, it should be formatted correctly, and you should use argparse to get the name of the input file from the user.  Your program should display, to the user, (1) the top 20 words and their counts, and (2) the top 10 puncation marks and their counts.  You do not need to write any results to an outfile.  Be sure to include the file(s) that you used to test/design your program.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse` to get the input filename (1.0 pt)
- [ ] Program uses a class-based approach and includes the requested attributes and methods (4.0 pt)
- [ ] Program correctly counts and displays both words and puncutation in any file (2.0 pt)


On line 6 of the src/Events/Event.php of this commit:
$xapi_type = 'http://lrs.learninglocker.net/define/type/moodle/';

In the ./src/Events/ directory, every event extends this variable. For the module viewed event, for example:
$xapi_type.$opts['module']->type

I'm making a new event (course completed), so mine became:
'course_type' => static::$xapi_type.$opts['course']->type
which I mutuated from the course viewed event (./src/Events/CourseViewed.php),

and it leads to a definition like this:
    "object": {
        "objectType": "Activity",
        "id": "http://abc.test.it/course/view.php?id=29",
        "definition": {
            "type": "http://lrs.learninglocker.net/define/type/moodle/course",

but the course activity type is already defined on the Tin Can API registry so the URI should point to that definition (http://adlnet.gov/expapi/activities/course) and not to this one (http://lrs.learninglocker.net/define/type/moodle/course).


Actually more than 12... I forgot to count the ones under "Source" that just lead to `https://mirrors.kernel.org/`.


singletonList()


btw with f#cker is the fc reason meant not the creator of the original commit lol


the jQuery library must be before any script that uses it. Some pages include scripts that will eventually be inserted into the @page_body macro. If the library is linked after this, it will break those scripts. I made a very specific comment about this in the comment message. This is the second time the library has been moved to the end of the page and is now the second time it has broken the scripts that rely on it.


If you need to write a script that relies on DOM elements, just wrap it in a jQuery doc ready callback.


I'll add `:realworld` to the PR I'm working on.


I see your point.

Even though this should not occur, it would change the UI:
- Currently: showing an empty viewlet (no callstack obtainable)
- With my change: showing the name of the thread

Whether I agree or disagree with what the correct behavior should be (showing the name of the thread if no callstack is available sounds like the right behavior to me), I will try to preserve the current behavior.


There is no need to store activity — it's state is used nowhere; also preferences is created in these activities and passed nowhere further.
The conclusion is that this class could be converted to util class with static method.


This is why I made the page footer macro, and I haven't experienced any scripts breaking when moving to bottom

On March 15, 2016 2:22:19 PM CDT, Charlie Hayes <notifications@github.com> wrote:
>the jQuery library must be before any script that uses it. Some pages
>include scripts that will eventually be inserted into the @page_body
>macro. If the library is linked after this, it will break those
>scripts. I made a very specific comment about this in the comment
>message. This is the second time the library has been moved to the end
>of the page and is now the second time it has broken the scripts that
>rely on it.
>
>---
>You are receiving this because you authored the thread.
>Reply to this email directly or view it on GitHub:
>https://github.com/idontusenumbers/JobMatch/commit/b2c316cba29d4f912fcd7d75c0ec9f645bbfef96#commitcomment-16716303

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.


Agree. BTW, this class is actually WorkerForBuilder**Base**.
I'll fix this in next PR.


This is not the latest commit in the PR. In the latest commit the naming convention issues are fixed.


It is, but the launcher uses 8. If you have opinions, voice them now.


Yeah I know, it was just nicer before, so wondered if there was an easy way to make it show something useful in that column.


2 things here. You want to return the node, not the value of the node (see assignment description). Also you don't really need to have a `found` flag here. The `return` statement is handy, because it breaks any kind of loop or conditional statement you may be evaluating within your function.


Aha! Good to know! Don't you love javascript? :D


There is no information about what is used as a regex so #group(1) could lead to AIOOBE


nope im not


Used the -N option insteads of -O and used || to have bash fallback to subsequent statements if the first fails. so,

statement1 || statement 2 || statement3

if statement 1 fails, it is ignored and statement 2 is ran. If both statement 1 and 2 fail, statement 3 will be ran and the first two ignored.

Submitted a pull request.


If a `cursor.getString(2)` returns `null`, it's NPE


reverted in 43d11b6


foo


What about handling it with `errors.onUnexpectedError`?

Do you see a circumstance in which a caller would want to handle the error in a different way, and hence the need to expose the promise? Right now, it's used in `debugService.setBreakpointsForModel` and `debugService.toggleBreakpoint`.


Really clean formatting for the most part! I loved looking at your task 3, it was soooooo much shorter and more efficient than mine!

Task 1: For some reason you forgot to do the unique words in chapter 2. Otherwise it functions correctly and does output (though different than mine...but who knows who's cleaning strings correct, probably none of us) what appear to be logical word count values.

Task 2: Some formatting issues with spaces between functions in this task, and also, not really wrong, but you imported modules you never used....could be confusing in longer code. Otherwise works well. I also loved your descriptions of your argparse command line input requirements. It was really clear what to input.

Task 3: We have different sum values for task 3, but honestly your code looks like it should work correctly and I dunno which of us is incorrect. But I did run a recheck on them and I got this for a value....
![capture](https://cloud.githubusercontent.com/assets/16652287/13791239/2072214c-eaba-11e5-81ce-82f0c8f65c69.PNG)

Though again, who knows. But your code looks solid to me so its hard to say what could be wrong with it, I can't find it anywhere and I don't feel qualified enough in map() to know if thats the cause of it or even if in fact its the wrong value.


only directives and pipes. see issue https://github.com/mishoo/UglifyJS2/issues/999


#5507-Polygonal-and-TriangularDoor-have-the-same-value (rraustad) - Win64-Windows-7-VisualStudio-12: OK (2139 of 2139 tests passed, 0 test warnings)

![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)


index.ftl has some script in it that will break. It's pretty minor, just default focus for the username field.

I can understand why you would want the non-library scripts to be on the bottom, even if you dont want to wrap them in a DOM ready callback. The jQuery library doesnt access any DOM elements though, so there's no harm in puting it up in the head. You can see the main.js file remained in the bottom before you moved the script links. It's just the libraries that were moved into the head.


Returning Boxed type just to provide "null" value is not very good for me. I'd go with either
1. Use checked exceptions for this situations, or
2. Pass sms as argument here and modify it only if match succeeded.


demo pyme


## Task 1

There are 2 chapters of Darwin's Origin of Species in this repository within a directory entitled `task1-files`.  Write a program that uses `argparse` and `glob` to open and read these chapters and some other code to get a list of all words in each chapter.  Then output, to the command line, a pretty-printed list of the following:

* the total count of words in Chapter 1
* the count of unique words in Chapter 1
* the total count of words in Chapter 2
* the count of unique words in Chapter 2
* the count of words in Chapter 1 that ARE in Chapter 2
* the count of words in Chapter 1 that ARE NOT in Chapter 2
* the count of words in Chapter 2 that ARE NOT in Chapter 1

Your program should contain a main loop and an ifmain statement, it should be formatted correctly, and you should be sure to use argparse to `argparse` and `glob` to read in the chapter files.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse` and `glob` (1.0 pt)
- [ ] Program functions as requested (3.0 pt)

## Task 2

In the directory `task2-files`, there are a lot of files.  Write a program that uses `argparse`, `glob`, and `os` to copy **only** the fastq files to a new directory.  Your program should contain a main loop and an ifmain statement, it should be formatted correctly, and you should take both the input directory containing the files you want to copy and the output directory where you will copy the files as arguments on the command-line.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse`, `glob`, and `os` (2.0 pt)
- [ ] Program functions as requested (2.0 pt)

## Task 3

In the directory `task3-files` there are 10 files, each containing a random selection of integer values.  Using a combination of `argparse` and `glob`, read these files.  Then, use `map` to compute the sum of all integer values across all files. Your program should contain a main loop and an ifmain statement, and it should be formatted correctly.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse`, `glob` (0.5 pt)
- [ ] Program uses `map` (2.0 pt)
- [ ] Program functions as requested (1.5 pt)


You could set the `OnClickListener` in `onCreateView`; if the `Button` is `GONE` the clicker will do no harm. That way it wouldn't be repeatedly added if the list were to alternate between empty and nonempty as crimes are added and deleted.


class should be static and immutable


You might consider the more general type `View` rather than `LinearLayout` so that the XML could be changed without needing to change the Java.


@malkomich Due to this fact we/I disable the autoformat of the IDE and also disable the optimization of imports.
Sooner or later we should specify one common autoformat within AndroidStudio and force to use this for all devs, but until then we should avoid autoformat.


Unity ma swoje funkcje, których raczej warto używać zamiast Console.WriteLine
http://docs.unity3d.com/ScriptReference/Debug.Log.html


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


Assuimg you're talking about line 193? Yeah maybe it should return the due date of the submittedAssignment. What I was thinking though is that maybe it should just error in that case since if we can't find the assignment in the cache then something has gone terribly wrong....


jcr6 module brought back, and added webread module although it's not fully functional yet.


Trying to have fewer silent failures in things


Jak już mamy referencję na mazePrefab i jest on jeden to nie wiem czy jest sens robić go poprzez Instantiate czy nie lepiej po prostu zrobić go jako obiekt na scenie na stałe i przekazać mu referencję do tego obiektu przez pole public.


One more class where storing activity is not needed


Sure! I wasn't sure whether this would fit into the platforms that you wanted to support, so hadn't proceeded with a pull request. But yeah, this adds support for `react-native-web`, works like a charm :)


`/etc/hostname` is empty? That's odd too. It should read at least `raspberrypi` unless you re-set that in your `installer.config.txt`


although yeah I probably shouldn't break out of the whole function I guess....


I don't see any use of `noCrimeMessage`.


`activity.getContentResolver().query(Uri.parse("content://sms/inbox"), <query>)` could be extracted to a private method


Any checks for nullability?


sdsg


Consider using a more general type, `View`, rather than `LinearLayout` if you aren't making use of any features specific to `LinearLayout`. That way you have more flexibility to change the XML layout without needing to change the Java code.


@mjankowski I had the same problem and fixed it with a `bundle-audit update`. Hope this works for you too.


Please add spaces after commas.


We should add a comment indicating which types are acceptable Object values.

JSONObject.put(String,Object) says "Object: a JSONObject, JSONArray, String, Boolean, Integer, Long, Double, NULL, or null. May not be NaNs or infinities."


Sure.


Are KEY2 and VALUE2 unused?


It looks like you set this variable both here and again on line 51.


Wartość 0.1 to, wydaje mi się, trochę za mały krok. Jakby tak wziąć trochę więcej fiszek o tym najdłuższym tytule to takie sprawniejsze przewijanie staje się mało płynne i obciążenie procesora sięga nawet 90+ procent.


It was written to just blindly scan overall assignments. It turns out that we had an assignment this year that students submitted to, but was later removed, so I added that to the ignore list. I don't think there's really a need for a white list necessarily since I think students will only have submissions for slip day assignments? (do automated labs have submissions?)


If this works now, should we also update the other orders?


Por que o que? E exemplos só de codigos ou algo mais?

Em terça-feira, 15 de março de 2016, fsantanna <notifications@github.com>
escreveu:

> Por quê?
> Você pode dar exemplos?
> Também seria bom colocar um exemplo de código em que Perl se destaca das
> demais.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub
> <https://github.com/PaivaHugo/EDL/commit/0b0293f485d0cde24ad58e8894e789fa1e89495f#commitcomment-16714204>
>


Any way to not make this N^2?


you are streaming the data, but then writing it in a batch.
you can also do this in a streamy way:

https://github.com/dominictarr/pull-json-doubleline
and https://github.com/dominictarr/pull-write-file

``` js
var JSONDL = require('pull-json-doubleline')
var WriteFile = require('pull-write-file')
pull(
  sbot.createUserFeed({id: userId}),
  JSONDL.stringify(),
  WriteFile(path.join(exportDir, 'messages.jsondl'), function (err) {
    console.log('exported:', userId)
  })
)
```
use JSONDL.parse to get the data back from lines, json will be written separated by two lines, and will have indentation for readability.


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


There's no need to set the `OnClickListener` each time the `Button` becomes `VISIBLE`. You could set it once in `onCreateView` and leave it in place even as the list may transition back and forth between empty and nonempty.


@cycomachead also check this out. A new way to get multiple things at once


Trying to uniform the geocast output filenames to blender default ones


@deepu105 At least in linux this should be ``./gradlew``.


> It should read at least raspberrypi

by default the hostname is `pi` (see [here](https://github.com/debian-pi/raspbian-ua-netinst/blob/master/scripts/etc/init.d/rcS#L10))

and it's installer-config.txt :wink:


Иначе под виндой не собрается, appveyor грохается.


This needs a final else condition in case no proper parameters were passed. In that case it should output something like:
{"error": "Invalid parameters"}


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


not  apostrophes but  carrots symbols ^ in include/Tetris/Tetriminos.h
that were causing problems
I simple got rid of them you can replace them with apostraphes or quotes if
you want

On Tue, Mar 15, 2016 at 10:20 AM, Lance Booth <notifications@github.com>
wrote:

> Wait, what kind of problems do apostrophes cause in vim?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub
> <https://github.com/warlord500/SpaceT/commit/bf318aee9452ec2b3c3b5981f15ee9f79b4d9bab#commitcomment-16713802>
>


You've duplicated the code for adding a crime. It would be more appropriate to factor this code out. The "extract method" refactoring would make that easy.


I would change the id parameter to pageid so that it's less ambiguous.


Example OPERA ODIM H5 format data Ezeiza C-band SP Weather Radar


The issue is that we only have the frame that is focused, but not the thread the frame belongs to.
Hence we are iterating over all the threads, seeing if its callstacks contain the frame. So we are going to be getting the callstack for threads we don't need.

If I end up exposing a non-promise based callstack in threads, which would act just like a cache, that's what would be queried here. I'm not completely sold on that approach though.


I could upload the application, but it crashes while booting at the moment.


Task 1: Very clean formatting. The only suggestion I have is if you decide not to use a function rather than leave it in """ """ to keep it from running, just remove it. It makes things cleaner and less confusing.

Task 2: Interesting, you used shutil as well to copy files. I couldn't get that copy module to worked, but shutil worked easily for me...maybe you had the same problem!  One suggestion, be really clear what you want from you argparse. Originally I thought you just wanted a name for the output directory and when I did that it created a random file with the name that I gave it and the file was empty. What you really wanted was a path to a directory, so when I gave it a full path it worked well. Just make sure you specificy exactly what you want.

Task 3: Really nice. Interesting use of map(), wish I had thought of that.


Some people said they do, but no, I don't.


Our package.json specifies our engines as 4.x.x so this tests all of those versions.....

So you're saying we should also update that then?


This commit breaks the test. It's not /user/login that was broken it's /user/do_login and you go there only after entering a wrong password.


That should be just fine. :+1:


@richard-scott I will look into this, though, I tried it myself, and got a similar result...

it doesn't seem to be related to the changes made by my PR, so I guess it would be best to open a new issue for this matter... :thought_balloon:
I do have some ideas, though, on what might be the cause... just have to check (a.k.a. find the time to do so)


In general some improvements are made, but many new problems arised:
1. Data flow is not obvious hence some good understanding of concepts and further refactoring will be difficcult.
2. The code is very unsafe.
3. Some problems with java knowledge are present. It's quite hard to tell how bad it is because some concepts are not addressed at all and some android-related concepts are unknown to me.
4. I don't see how DB is filled for some kinds of data.
5. There are no tests.

To recap, I'd say there is very much work to do even at this point of time. However, I must say that the code itself is quite readable so it's easy to get the idea of what's happening. That's good.


ьь


Remove extra space.


afwImage, algorithms, afwDetection are unused.


I also just wanted to see what travis would do tbh. I'll need to make sure nothing breaks in v 5.x since I'm running 4.2.x


Add white space around '='.


Your `onResume` method seems prepared for transitions from nonempty to empty, but not for transitions from empty to nonempty such as would happen when adding a crime to a previously empty list.


__API Description file syntax error on line 301__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


Okay, thanks! Can't wait to run Impulse again, awesome work!


To nie zadziała. Argument `font` jest oznaczony jako var, który potem jest nadpisywany w pętli, więc tutaj nic się nie zmieni, no i wtedy zamiast `break` też powinno być `return`.
Poza tym idea chyba trochę sprzeczna z argumentami metody, jeśli sam określam jaki może być najmniejszy rozmiar to dlaczego miałoby to być 'siłą' nadpisane?

P.S. w Swifcie 3.0 nie będzie można oznaczać parametrów funkcji jako zmiennych.


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


You're right I didn't use any of your code as I rewrote the whole thing from scratch, but since you are part of Cookie Monster history as someone who has maintained it at one point, I still would like to give credit.


Notice that you've got the code related to the subtitle only in one of the two branches; your code won't properly set the subtitle's visibility if the list is empty.


Remove whitespace


I think this would be a better test if the previous call returned an endpoint (in addition to the `RECEIVER_ERROR`).


Does this Update function indicate that if you are not currently attacking then you reset to full stamina?


Bad merge


Can you be sure the parent has no other children?


This is one of the rare cases where you should actually use isset() instead of !== null. isset() tests to see if the variable exists and is not null. When you're not sure if an array index even exists, isset() is the correct way to test for it. This applies to lines 11 and 23 as well.


03 = obfuscation but no imputation

birth_date: 03 (from clarity.patient)
enr_start: 03 (first visit start_date)
enr_end: 03 (last visit start_date)
admit_date: 03 (encounter start_date for diagnoses, procedures, etc)
discharge_date: 03 (encounter end_date for diagnoses, procedures, etc)
px_date: 03 (observation fact start_date for procedures)
measure_date: 03 (observation start_date for vitals)

All other ‘NI’ for now cause we aren’t loading the CDM tables yet, e.g. rx_start_date, rx_end_date, …


das funktioniert schon, das FIXME bezieht sich auf das hinzufügen aller lehrer


@silverwind Would you please check the other new additions for the [review tools](https://github.com/blog/2123-more-code-review-tools)... I couldn't find a prev & next buttons, or the "view outdated diff". Thanks!


The patch looks great to me.  The only thing I would do is to add a blank line before the `return 0;` line that you added (line 2332).  I am compiling / smoke-testing it now.

I would simplify the commit comment, but that's just a personal suggestion;
feel free to take it or leave it.

> staging: unisys: visorbus: visorchipset_init clean up gotos and error paths

> Several error paths were not logging a message to s-Par during failure.
Error paths in visorchipset_init() were corrected so that they now all
do proper clean-ups.  This made it necessary to move the function
visorchipset_file_cleanup() above visorchipset_init() so it can be
referenced.


Eu até tinha feito dessa forma mas não sabia o porque que estava dando errado.


That's true. I didn't go for that approach because I think it's better to have an explicit flag rather than rely on special values (`null` --> `callstack` hasn't been obtained).

I do agree that adding `getObtainedCallStack` would save some refactoring due to promises.


I'd be happy to work with you on figuring this out.


Fiz a correção e além dessa reparei uma outra ao reler.
Valeu!


don't drink and code ;-)


Dobrze byłoby się trzymać jednej konwencji, wszędzie piszemy

        func nazwa(arguemnty) {
        }
To samo niżej w przypadków pozostałych klamer.


5521-EMS-InsideHVACSystemIterationLoop-AirLoop (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: OK (2024 of 2138 tests passed, 164 test warnings)

 * 165 tests had: ESO small diffs.
 * 240 tests had: EIO diffs.
 * 93 tests had: Table small diffs.
 * 66 tests had: MTR small diffs.
 * 108 tests had: ESO big diffs.
 * 35 tests had: MTR big diffs.
 * 50 tests had: Table big diffs.
 * 14 tests had: ERR diffs.
 * 1 test had: AUD diffs.

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


Yeah absolutely. I guess the code has only been auto-regen-ed for order 4? Either way, won't hurt to run it for the other 9 orders.


Looks interesting :)


BRANCH=next now contains the dtoverlay command (in fact it's in the master branch as well in readiness and as a way of reading the help), and the userland repo includes the source.


If your office hour this Friday is still free, I'll be able to come to that
one!

Beth
On Mar 15, 2016 2:50 PM, "Max Hailperin" <notifications@github.com> wrote:

> I'd be happy to work with you on figuring this out.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub
> <https://github.com/esherma2/CriminalIntent/commit/7096fc018a59c4757a129b4477f00306409678de#commitcomment-16716751>
>


5113_GHX_8.2-8.3_Transition (mitchute) - x86_64-MacOS-10.9-clang: OK (0 of 0 tests passed, 0 test warnings)

![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's my bad. I added this spec a while back. Made it point to actual rubygems/the actual gem because of the difficulty in simulating it. I can take another look at simulating it and in worst case moving it to the realworld specs.


just these lines?


Note that you've repeated on line 97 the same code as was causing problems on line 50, but meanwhile you still have the old code on lines 94-96 -- you didn't complete the refactoring by deleting the code that was moved into the method you were trying to call. (Speaking of refactoring: you could have avoided problems with this by using the "extract method" automated refactoring -- it would have done it right. Although I can also show you how to do it right by hand.)


Done, I will be pushing new branch soon.


Haha! Yes, bit of an experiment ;) I assume you can see what I'm trying to do here?


Bad merge here too


Phew, I was starting to think I'd broken something :wink:

Do you want me to open a new case? Or have you already done that?

I wonder if it can be something to do with DHCP? I have a reservation in my router to give my Pi the same IP address, but it does not set the hostname as my router is not that intelligent. I wonder if because it lacks the hostname if the DHCP client is setting the hostname to an empty string then perhaps that's the issue?


This looks nice but give me the error:
```
   Compiling openzwave-rust-example v0.1.0 (file:///home/work/fxbox/openzwave-rust-example)
src/main.rs:225:61: 225:84 error: mismatched types:
 expected `openzwave::manager::NodeBasic`,
    found `collections::string::String`
(expected enum `openzwave::manager::NodeBasic`,
    found struct `collections::string::String`) [E0308]
src/main.rs:225                                  node.get_basic().unwrap_or(String::from("unknown")),
```
If we added an Unknown entry into the enum then we could use something like this:
node.get_basic().unwrap_or(NodeBasic::Unknown).

I'll try the earlier suggestion as well


Among the examples of other places we do something different, I am most familiar with `PUT` vs `POST` since I am the one who introduced the structure there to avoid code duplication. I consider that there are are couple differences though.Firstly, there is no "type" for which the domain is only `PUT` and `POST`, but not `GET`, `DELETE`, etc. I guess we could create such a type if that was the only issue. Secondly, there are actually some small differences between the semantics of `POST` and `PUT` and the tests in a few cases look at the action that was passed in as argument and assert slightly different things.

I certainly see the similarity though and I don't think it would be absolutely incorrect to go either way in any of these places. But unlike in `DataServiceSpec`, here the nature of the `Codec` is truly independent from what is being asserted, so I consider it slightly more elegant to use a property based style.

From a code duplication standpoint. I think it makes sense either way to provide a `Arbitrary[PathCodec]` in the `scalacheck` module for downstream clients, so by using that instance we only have to remember one place where to add any additional `PathCodec` defined within `core` instead of two places.

If I haven't managed to convince you, I think, as you suggested, we should ask for some quick feedback from the rest of the team since it sort of boils down to a stylistic concern and it's probably not worth spending too much time debating the merits of both approaches :smile:


In Dutch, oh well.


5521-EMS-InsideHVACSystemIterationLoop-AirLoop (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1582 of 1582 tests passed, 0 test warnings)

![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.86%25-green.svg)


Hmmm.... I'm getting build warnings... it seems like they MIGHT be caused by this patch...
```
  CC [M]  drivers/staging/unisys/visorbus/visorchipset.o
drivers/staging/unisys/visorbus/visorchipset.c: In function 'bus_epilog':
drivers/staging/unisys/visorbus/visorchipset.c:912:21: warning: 'pmsg_hdr' may be used uninitialized in this function [-Wmaybe-uninitialized]
  if (pending_msg_hdr->id != (u32)cmd_id)
                     ^
drivers/staging/unisys/visorbus/visorchipset.c:962:35: note: 'pmsg_hdr' was declared here
  struct controlvm_message_header *pmsg_hdr;
                                   ^
drivers/staging/unisys/visorbus/visorchipset.c: In function 'device_epilog.constprop':
drivers/staging/unisys/visorbus/visorchipset.c:951:21: warning: 'pmsg_hdr' may be used uninitialized in this function [-Wmaybe-uninitialized]
  if (pending_msg_hdr->id != (u32)cmd_id)
                     ^
drivers/staging/unisys/visorbus/visorchipset.c:1025:35: note: 'pmsg_hdr' was declared here
  struct controlvm_message_header *pmsg_hdr;
                                   ^
```


This worked (directly in the println!:
```
node.get_basic().map_or(String::from("unknown"), |basic| format!("{}", basic))
```


# Just checking if it is working


* Morôni Augusto Dourado Rabelo -> [Visite meu Git!](https://github.com/moroniaugusto)


Ajuster la doc. On utilise argsObj  (un objet) plutôt qu'un array.


Cool! Thanks for the heads-up.


Oh yes, I can see what you're trying to do.

Not sure about your log output - might just be nicer to get it so that the `elapsedTime` is provided to *observers* of the profiler. I have half of an idea, but I don't want to step on your toes here, so will be interesting to see where your experiment leads to!


test


On Tue, Mar 15, 2016 at 2:51 PM, esherma2 <notifications@github.com> wrote:

> If your office hour this Friday is still free, I'll be able to come to that
> one!
>
> Beth
>
>
Yep, I'd be glad to see you then. -max


La fonction "obtenirCouchesParNom" ne semble plus là dans la dev ? @stetou  et j'ai commencé a m'en servir.


yeah that would be the better candidate for renaming, since that isn't even the same kind of thing, whereas the other ``args`` are just regular E2 tree nodes.


Set peace in all Human brains


I haven't actually tested this one, but I'm fairly certain it needs to be 3.  The strlen() doesn't include the null terminator, and we're about to put subsys in between two forward slashes.  So + 2 (for the forward slashes), and + 1 for the null terminator.


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1598 of 1598 tests passed, 0 test warnings)

![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)


:+1: release!


Ah ya, completely forgot. I'll resubmit the PR.


ok


Estaría de sobra. El partner es ese mismo. self.partner
Y lo dicho anteriormente, si haces un enlace inverso de partner a employee, te evitas todas las líneas de búsqueda. O como mal menor, si metes la función una única vez en employee y le llamas desde los diferentes puntos, te evitas también repetir las líneas una y otra vez.


Agora vai fi


No haría falta


dosen't look like the pull request adds this image


amazing


Good catch! Yes, it should be "core2-32"


You have gotten to the core of the question: I do see us as declaring what we support in configuration files like this one. I don't see a compact, robust way of separating what this command says from our commitment to support what it does. I'm open to suggestions.

If we do support something older, then I think a user has a right to be upset if they report a bug and we do not respond to it. If something older works, and later stops working, I think users have a de-facto expectation that we will help them in their upgrade. That makes version changes asymmetric: going from newer to older entails significantly more risk than the other way around.

And this is a double standard: why should probcomp bear the cost? Rather than making this change to probcomp on your weekend, you could have helped NetBSD fix their... oh wait, NetBSD's latest supported version is 7.0, with a gcc that supports c++11. So again, why insist on out-of-date software?

Picking battles, this is a bad use of our time: NetBSD 0.1% of web servers; that + old version, + will not upgrade gcc: tiny minority. This hypothetical user's contribution detracts from our focus and velocity in pursuing actual probabilistic computing (not packaging), and it sets up the potential to get in our way when we encounter a user for whom it doesn't work.

Contributions that increase the potential use of probabilistic computing are very welcome. For example, I very much welcome and appreciate the larger contribution that this one line is a part of, because it will make installing from source significantly easier on any system. Thank you again!

Because I do wish to support you personally, because you assure me that the risk in this case is minimal, and because I don't really want to have more of this discussion, LGTM. Go ahead.

(Or upgrade your system already, e.g. for the security enhancements that come with blacklistd(8).)


Bumping `REL` in a bit.


also `.ua` and `.kz` are valid


Per style guide, the `tls_configs` should still be a pointer and not a reference (we try to avoid non-const references in function signatures). It should also be called `tls_configs_out` because it's an output parameter.


Tests revealed this not to be the case: echo ((int)"xyz" == "xyz" ? "fail" : "pass");