This list doesn't include the "wal" or "meta" buckets that can also appear in b1 shards (according to https://github.com/influxdata/influxdb/blob/0.10.0/tsdb/engine/b1/b1.go#L123-L130).

If nothing else, not excluding these too is going to lead to spurious statistics.

My original heuristic used the presence of , in the bucket name as a filter since it is true that all buckets containing series data include a name that does include ,


不需要指定水印图标路径了喔 ~


Please don't name your method names gogogo()

```
def main():
    pass

if __name__ == "__main__":
    main()
```


Works for me now.
I'd still ask for `assert(0 <= firstNonZero && firstNonZero <= 3);` or something similar to explain the prediction mechanic in code.


TestCustomCommand (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (977 of 977 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-28.31%25-green.svg)


111768812_VRFNoLoadOAFlow (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1508 of 1508 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.69%25-green.svg)


SM now can make. A few things:
1) there is no test script, so I can't test whether the code is correct or not;
2) try not to use \t as indent;
3) try to make the code structure more clear;
4) util/join.cuh defines functions under gunrock::util space that only used by SM, and named as mark, update, debug_0, debug_1, etc. If these functions are only used by SM, put them in gunrock::app::sm space; if they are indeed utility functions, please give more descriptive names.


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


A cluster must have at least three nodes running the consensus service (consensus or hybrid nodes), but it can have more.


checked


Yeah, however in the wild I don't believe anything is normally between the two, since the idea was to prevent older browsers from having issues with HTML content embedded in the JS.


Instead of using a rule for identifier, we should be able to just use identifier:
```
    | GRANT
        (privilege (',' privilege)* | ALL PRIVILEGES)
        ON (TABLE)? qualifiedName TO (grantee=identifier | PUBLIC)
        (WITH GRANT OPTION)?                                           #grant
```


For privilege, @martint would prefer that we allow any identifier for the privilege.
This allows the analyzer step, in `GrantTask`, to throw a cleanly formatted error
instead of giving the user a parse error.  Additionally, we don't ned the `#privilegeNode`
tag since Antlr, will just create a `Privilege` object.   Something like this:

```
privilege
    : value=(SELECT | DELETE | INSERT | identifier)
    ;
```


Thanks for your efforts.
Some feedback: Commits of this size are a code maintainability nightmare. If someone wants to use git blame / git bisect to look up why a certain piece of code exists, it will be impossible without a proper Git history.


@sgpyc Please add these as issues and assign them appropriately so we can keep track of them and they don't get lost.


Then we don't need this rule.


All of these should be added to non-reserved words.  Also with the change to permission above, we don't need to declare `UPDATE` yet.  We should add that soon, but that would be a different pull-request.


removed `q` as a dependency @stevemao


@gabrielsaldana

From the email.
>For now, I’ve avoided this by special casing the directory name.  Currently all tests pass, but it would really benefit from your feedback.

I would have expected to see the special case removed.
https://github.com/ohsu-computational-biology/server/blob/g2p/ga4gh/datarepo.py#L191

Can you remove this special case and then re-test?


This code just tries each `jwk` until it finds one that works; might want to adjust this to follow the advice that @wemrysi has given in the thread above, but it probably works as a first cut.


We can switch `prestoIdentity` to a simple `String` and rename it to `grantee` to match the grammar.


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


I'm so sorry @defuse I didn't want to push to this branch at all!


This should be `String` also, so the analyzer step can throw a clean exceptionm


The method and the next shouldn't be needed


As from above, I'd make this `String` and rename to `grantee`


As above, I would make this `List<String>`


I think you missed a minus at line 1301  :D


This shouldn't be needed


Use `FRC.constantPeriodic`.


Why do you need to have these variables?


@sgpyc Surely will have the above things done in dev-sm first and then merge back to dev branch. Thx for listing the problems.


Added comments in commit https://github.com/jaredmoo/elastic-db-tools/commit/f7e6e8edf595062e17ee83b4d0697ee050fe2af5


:ok:


this doesn't do anything, unfortunately


@mzhurovich : Yep, `bool ReplayTransaction(...)` -> `void ReplayTransaction(...)` and throw :arrow_up: ?


Agradecimento ao membro **Spell Master** pela contribuição.


I'd say it's safe to accept any input type here, for the sake of shorter code and faster compilation time.


I actually like it being part of the controller export, because I could use the same action name in different contexts.


but, doing a custom request serializer seems like unnecessary overhead at this point, since request serializer are more suited for requests w/ custom query parameter or request payload logic—not to build the request URL itself (which is an argument of the `GET` call on the manager)


done


so to answer your question—sometimes... if there are things they do that can go wrong—but we should generally to keep fetchers as lean as possible. MW API doesn't play so nicely w/ the conventions AFNetworking was designed for.


The reason I had to implement was simply to satisfy the use on process code. I'm aware the replacement is not atomic at all (I should have commented that on the code.)

My vote is to remove the atomic usage (and the replacements) unless we have evidence of concurrent access at this point.

If necessary, I agree disabling interrupts might handle the only situation I can imagine concurrent access on M0 (exception handling). We could also do some research on how other embedded firmware manage this.


Not sure if I'd add this as a breaking change, since it's not a breaking change. Unless you preemtively put it under v4 and have a note at the top saying they are marked obsolete from v3.3, but can be still used?


Done.


I mean the way you pass `TransactionMetaFields meta_fields`.


This is more complicated than it needs to be. I would just recommend multiplying `leftOut` and `rightOut` by `maximumSpeed`. So if `maximumSpeed` is .25 for example, full throttle will make it run at .25 speed.


typo: "resources"


name this `isGrantObjection` or `isWithGrantOption`


FixGSHPweirdOutputOfAsterisksEtc (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (2086 of 2086 tests passed, 6 test warnings)

 * 6 tests had: AUD 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)


There should be an odd number nodes running the consensus service in a cluster.
The number of consensus services that can fail before the cluster is degraded is ⌈n/2 + 1⌉ where `n` is the number of consensus services in the cluster.
Thus, even numbers of consensus services offer no additional redundancy or resiliency.


@pvorb @iMiiTH +1


Looks like `candidateKey == null` is a possibility, therefore nothing would be removed
```
if (candidateKey != null) {
  map.remove(candidateKey);
  counters.peers -= num_peers;
  counters.torrents -= 1;
}
```


This should happen automatically at some point.


technically i think it's safe since it's reading immutable properties, but i'll trade ruling that out for a minor CPU hit when this promise resolves


Implemented Issue #2 Auto Close of UI after auto upload is finished.


it's meant to convey that you _want_ a non-null value. it _might_ be `nil`, or another type, in which case you get `nil` back and an error describing what went wrong


In the commit message, yes. This is done by all the other drivers: you don't want to fail the initialization of the driver just because you'd have to wait on the lock.


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


*chuckles*

👍


> System.Collections.Generic.KeyNotFoundException: The given key was not present in the dictionary.
>    at System.Collections.Generic.Dictionary`2.get_Item(TKey key)
>    at DamageMeter.Abnormality.ApplyEnduranceDebuff(Int64 lastTicks) in H:\c\ShinraMeter-master\DamageMeter.Core\Abnormality.cs:line 74
>    at DamageMeter.Abnormality.ApplyBuffDebuff(Int64 tick) in H:\c\ShinraMeter-master\DamageMeter.Core\Abnormality.cs:line 56
>    at DamageMeter.AbnormalityTracker.DeleteAbnormality(EntityId entity, Int64 ticks) in H:\c\ShinraMeter-master\DamageMeter.Core\AbnormalityTracker.cs:line 105
>    at DamageMeter.AbnormalityTracker.DeleteAbnormality(SDespawnNpc message) in H:\c\ShinraMeter-master\DamageMeter.Core\AbnormalityTracker.cs:line 88
>    at DamageMeter.NetworkController.PacketAnalysisLoop() in H:\c\ShinraMeter-master\DamageMeter.Core\NetworkController.cs:line 181
>    at System.Threading.ThreadHelper.ThreadStart_Context(Object state)
>    at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
>    at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
>    at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
>    at System.Threading.ThreadHelper.ThreadStart()


Okay, this looks pretty good. `Except that it should look like code`
Could add a [link](GitHub.com) if I want, even in the comment.


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


oh... it was already blocking (I misread it as nonblocking)... `_serial->sem_take_blocking()` is exactly what we are doing here.


You know you have to submit this as a pull request, right?


if they are declared as `static const` (either local to the file or in a class), yes.


The mailer is working OK on development (localhost:3000) but emails still aren't arriving at fixedgearnation.contact@gmail.com.  Otherwise going to Contact Us on the nav bar in production (Heroku) is failing.  Some kind of db:migrate issue, not sure.


why not just have data.payer set to the payer?


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


maybe when requester_pays we store two entries? one with the user set to the owner, one with the user set to the payer. Have an extra field in data that specifies whether user was accounted for the operation


a bit hacky. I'd rather just have an extra field.


Puzzled now. If `assert(0 <= firstNonZero && firstNonZero <= 3);` is not true for any value of `BigDigitBits`, I fail to understand the prediction.


This commit has already been applied with no 0 initialization.


Line 19 and 21 do the same thing.


Just realized how spaced out I made the table xD


@mcmonkey4eva ...how does one aquire a queue with no script entries?


FixGSHPweirdOutputOfAsterisksEtc (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2118 of 2118 tests passed, 6 test warnings)

 * 6 tests had: AUD 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)


the /ex command


The difference is that get checks if the keys exist. But in this particular case, the substates are created in advance, so it does not matter. Your version is certainly simpler, so I have changed.


Static import (and USER also)


Data or Hybrid nodes


What is the recommended commit to use if I want to stay on FreeRDP 1? There is no stable-1.2 branch...


the grantee should just be a string


If table is not defined, hrow will be 0. Not sure if it's always OK.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 15707](https://teamcity.mesosphere.io/viewLog.html?buildId=134762&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:10


FixGSHPweirdOutputOfAsterisksEtc (JasonGlazer) - 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)


;-; the /ex command doesn't run a script entry? what does it do


@komagata
ああ、なるほど！ ありがとうございます、別々のことだと誤解して悪戦苦闘してしまってました。

@machida さん、 https://github.com/fjordllc/interns/commit/1d57cd92af1ca06364e49792d6b7670b27436689 で
> フォーム変更検知をDRYにする
> jsプリフィックスを付ける

に対応させて頂きました。ご確認お願いします。


...and one low resource node that acts as a consensus node.


I actually removed the initialization since it's not necessary


Please leave your comments


accidentally pushed here, no easy way to undo a push, but working on it...


Idk @BlackCoyote tried it with /ex and it no work.


@adaur i applied all your changes to my branch:

https://github.com/fernandog/SickRage/commits/manual-search

can you make sure everything is ok? If so, I will update SickRage branch and you squash commits so you can be the author


I'm so determined to fix my root I'm almost there I'm really enjoying myself like learning new things can u help me please


I think we should figure out how to integrate this into the existing `loadTablePrivileges` code.  I originally just made grant another Permission, but instead we should make a class to wrapper Permission and the grant option flag.


I would probably just add the negation in this line.


@dinoboy197 `Main profile` is a localized string (`Hauptprofile` in de_DE for example).


I see I misread the diff.


Because someone could submit a PR that abused the root permissions and Travis would run it. Also, unknowing users or maintainers might do it. The change I made is a much more secure middle ground.


I realize this only has one referrer in `ActivityApiController` but it doesn't feel quite right to exclude locked activities _before_ returning from the repository class. I'd rather see those locked activities filtered out in `ActivityApiController`.


please Bear with me I'm not dome nor a slow learner but can u please help a lil more I know the drag and drop I'm a fast learner may u please help me a lil more


You can switch on the enum directly instead of the string value.


Is it actually possible for `other_filters` or `read_length` to be `None`? I thought everything had a default.

They could also be `0` or `""`, which is still `False in Python. So I'm curious about that case. In the test (other file), I see `( length >= 0 AND length <= 1400 )`. Where does the `length >= 0` part come from? Not here, obviously, since `0` is `False`.

And what would happen if you always ran this code, instead of only when `read_length` is `True-ish`?


Might as well state the read_length, no?


This is a compile test. It can't check if your cpu has avx2 or not. What you should do is check if your compiler/assembler support avx2 instruction.


:) yeah - still a few things to do before officially releasing 1.0.2


Se debe de agregar el campo  $table->softDeletes(); a la tabla user mediante una migration `php artisan make:migration add_deleted_to_users_table --table=users`


Tyler Nethery, tylernethery1994@siu.edu


i just played with this on node.  if table is not defined hRow will be undefined.


FixGSHPweirdOutputOfAsterisksEtc (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (2086 of 2086 tests passed, 6 test warnings)

 * 6 tests had: AUD 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)


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


Oh, I get it now. The granularity of prediction is not `3` but `BigDigitBits / 3`.

The bits come in chunks of length `BigDigitBits`. Each unit of `data.length` contains from `0` to `BigDigitBits / 3.0` new digits.

So, the correct assertion would be like `assert(0 <= firstNonZero && firstNonZero <= BigDigitBits / 3 + 2);`, one of the `+ 2` for the `predictLength = 1 + ` and perhaps (?) the other for the rounding. Does that sound right?

Anyway, sorry for the noise on the issue.


amen


What we need to do here is runtime test if the cpu support avx2 or not.


Service satisfaction is part of it. I try to focus on helping achieve a goal particular to the project.

>My answer about memcpy hasn't changed.

Alright, I'll look into it. My biggest issue was dealing with cases where ROM is 16bit aligned and RDRAM was 32bit aligned or vice versa. The code could definitely use some reviewing on my part. I'll also profile and see how much faster memcpy is, in the case where everything's at least aligned by 32 bits.

>This feeling would be fact if the implication that memory staying in little endian, itself, was beneficial.

I'm sure there are pros and cons to both. For one, plugins would need to be rewritten. Then there's the question of whether performance will be better or worse. It also has to do with how long it would take to transition into big endian. If it doesn't happen for several months or more, then I think it's fine to have some temporary optimizations.


what exactly does sending them to clip board do and the problem is on the reboot there seems to be some sort of block and won't fully reboot


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1536 of 1536 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)


@aggelosQQ what am I supposed to "properly" do?


@aggelosQQ cachedview.com


@rkorytkowski it seems to me that the correct number is actually 8, but there were a couple of user-created apps on the home screen that shouldn't have been counted. Someone must have just disabled one recently, leading to tests breaking.


This is wecome


test-json be removed from gitignore too.


thank u very much I'm really getting the hang of this and really having a good time learning have a nice day


For the sake of getting it done I was ignoring everything that wasn't relevant to add as a filter, like readlength >= 0. I'll make another pass and improve the code and testing when I know that this works.


Already in previous text, but "is was set".


@smarterclayton I know this is awful, but do you hate it so much that I need to have a different approach before 1.2? I didn't want to rush a change to your pretty interfaces.


TestCustomCommand (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1529 of 1529 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.7%25-green.svg)


TestCustomCommand (jasondegraw) - 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)


lgtm


Agreed, but using which implementation? I don't want to import a whole burrow


Yep.


euler is a great way to learn a new language for me.

btw, does php has something like `reduce`? I write js mostly these days so i would probably write something like this:

```
arr.reduce((prev,curr) => {return prev + (curr % 3 === 0 || curr % 5 === 0 ? curr : 0 )}, 0)
```


Why this change?


Excited to see what changes will happen with the API.


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


I don't know a lot about Travis security concerns, but couldn't people do that anyway once your Travis setup allows sudo? I am likely just unaware of some feature that prevents this, else you'd have to set up private forks to test this kind of stuff (and Travis would not support sudo for public repos).

Anyway, it was just a minor bother, the important thing is this aspect gets tested, somehow.


Oops, scrap that. You do have ->valid() check.


What error?


and

> System.NullReferenceException: Object reference not set to an instance of an object.
>    at DamageMeter.DamageTracker.Update(SkillResult skillResult, Int64 time) in H:\c\ShinraMeter-master\DamageMeter.Core\DamageTracker.cs:line 305
>    at DamageMeter.Abnormality.Apply(Int32 amount, Boolean critical, Boolean isHp, Int64 time) in H:\c\ShinraMeter-master\DamageMeter.Core\Abnormality.cs:line 47
>    at DamageMeter.AbnormalityTracker.Update(EntityId target, EntityId source, Int32 change, Int32 type, Boolean critical, Boolean isHp, Int64 time) in H:\c\ShinraMeter-master\DamageMeter.Core\AbnormalityTracker.cs:line 152
>    at DamageMeter.AbnormalityTracker.Update(SCreatureChangeHp message) in H:\c\ShinraMeter-master\DamageMeter.Core\AbnormalityTracker.cs:line 159
>    at DamageMeter.NetworkController.PacketAnalysisLoop() in H:\c\ShinraMeter-master\DamageMeter.Core\NetworkController.cs:line 147
>    at System.Threading.ThreadHelper.ThreadStart_Context(Object state)
>    at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
>    at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
>    at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
>    at System.Threading.ThreadHelper.ThreadStart()
> System.Collections.ListDictionaryInternal
>
> Void Update(DamageMeter.SkillResult, Int64)
>


The Index originally returned the Message Model opposed to the Message ViewModel or in this case the ForumViewModel.  This obviously caused conflict, the update returns a ViewModel


Might be better to be more specific when specifying the table name  to validate?


Is 1.4 almost ready to go?


@jleclanche

Note that I ended up doing this because I wasn't sure what the semantics of entity.entity_id is.

If you tell me that entity_id is unique across entities, then I'll probably replace this with a dictionary on entity_id -> entity for the set operations (since the entity_id keys themselves can be used).

As an aside, entity equality right now is on the card ID, which seems wrong; in a lot of places in fireplace we are using entity1 is entity2 when we probably mean entity1.entity_id == entity2.entity_id. Fine to look into cleaning that up?


indentation issue


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


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1536 of 1536 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)


Same comment/question as on the same case of this earlier.  We shouldn't make changes like this just for the sake of boosting our coverage numbers.


Nit: These statics should start with ```s_```.


Is there another test case for any exclude table filter option?


Thanks!


@atavism Added this to .gitignore so we don't accidentally check it in.


We may need to keep one test for testing partition tables?


@atavism Mobile doesn't need BNS_CERT_PASS, so I refactored this a little.


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


To further reduce the risk of checking in the keystore, I made this a symlink.


There's a lot of duplication in all of these test data sets.  Could they be produced programmatically instead from a single copy of the test data?  I think that'd be easier both to comprehend and to maintain.


@medariox @supergonkas

![manual-search](https://cloud.githubusercontent.com/assets/2620870/13024840/8ee841c4-d1e2-11e5-9166-974fe96087b7.png)


@atavism For convenience, I added a link to the correct file in too-many-secrets.


This is used for the `/transfers/:id/state` endpoint


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


FixGSHPweirdOutputOfAsterisksEtc (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (984 of 984 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-28.54%25-green.svg)


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1552 of 1552 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)


Oh, I was asking more academically, not critically.


@komagata @machida
すみません、 `app/assets/javascripts/warning.js.coffee` の `$('body.' + $('body').attr('class'))` という表記が冗長だったので `$(this)` に修正しました。
こちら https://github.com/fjordllc/interns/commit/01f69f1126888e2994a8422652057e5161d921de の方をご確認頂ければと思います。よろしくお願いします。


Do you think we need to keep this definition? I guess we don't know if everyone switched over to the new layout with the buttons across the top.


I don't think so, since the map is non empty and does not contains the `hex` key. If anything, the `if (hex.equals(kv.getKey())` is unnecessary.


develop (Myoldmopar) - 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)


Issues filed as #117 to #129 (including the issues for dev-sm branch).

@jowens Thanks for the pointer to GNU indent, I tried it seems need to figure out the settings for current Gunrock's coding style. The output of default GNU style on gunrock code is not so nice...


Incorrect ticket number referenced, should have been https://github.com/obophenotype/mammalian-phenotype-ontology/issues/2139


>If it doesn't happen for several months or more, then I think it's fine to have some temporary optimizations.

Well what you just said, was that how long it takes to transition into big-endian is one of its cons.

Then, after, you just said, it's fine to have _temporary_ optimizations?  Despite the fact temporary optimizations like this make transitioning into big-endian harder and take even longer.  Sorry but I don't see how those two statements mix.  That's effectively aiming for them to be *permanent*, not temporary.  If they're not permanent, why do them?  That's just a con.  Making it temporary here means accepting the fact that big-endian will take longer, then finally moving to big-endian anyway and wasting time twice instead of once.

Then again, none of those cons against big-endian are really cons towards the thing itself.

* How long it would take to transition into something doesn't specify any disadvantage or con into having said thing done.  That's a pragmatic measure of human time investigation, not strategic correctness.
* Plugins need to be rewritten?  See above.  Is not a con to the emulator itself, just plugin authors.  That's besides the fact that code is supposed to be kept maintainable, so compiling to little-endian is still easy.
* The question of whether performance will be *conspicuously* worse is valid question, yes.  It has to be noticeable.  Adding a few tiny micro-instructions for "slower" performance is not a relevant con when you're weighing that against making the emulator functional, working, and accurate.  And before you contemplate how big-endian will make some parts slower and some parts (like this file) faster, I remind you that those are ROM-specific considerations based on the liklihood of Nintendo's development practices.  Really not the same thing as assessing performance itself being better or worse, if it's both.


Nice catch. Fixed in 6aa08432dddc94814c64eca08b1b6f370520172c.


Should the strings be wrapped like a field actually? Just want to make sure I understand what's fully expected in this array situation?

```js
cats: [
  { value: 'lion' },
  { value: 'panther' },
  { value: 'garfield' },
  { value: 'whiskers' }
],
```


It should be image-banner.p**ng** instead of image-banner.p**hp** according to common/server-banner/.htaccess.
`RewriteEngine on
RewriteRule ^(.*)\.png $1.php [NC]`


If an assert fails, an exception will get thrown, and the culture will never be set back to the original.  The resetting of any thread state like the cultures should be done in a finally block (here and any other places it shows up).


I don't understand where and why this is needed... Could you please provide an example?


> Did you do a full project-wide Search&Replace?

No, I just ran `M-x whitespace-cleanup`.

> (I think it should just be done once and for all)

Ideally, yes, and there should be also a coding style check on Travis for catching this stuff.


I'm not sure how typical that is.

Anyway, I'm not necessarily *complaining*, I just find it a bit weird. I do tend to find it actually hurts readability in some instances, and (overall) doesn't improve readability *enough* for it to be worthwhile. Consider that people who *aren't* using three emacs'es side-by-side! :)


For Mario:
Create pull request


Do you know if there's an issue for the automated check bit? This is one of those bite-sized things that I might actually get around to implementing in my Copious Free Time(TM).


Oops


fdfd


After trimming down, these acceptance tests should go in features/txt-font-props.feature rather than their own file. They can be added at the top so they're easy to find while we're working on them.


Maybe, the module is quite large as it is.


I don't think so, you can open one if you want.


Woot!


fail


Agreed... there's plenty of room to decide and fine tune how this new feature can used, but the forum will be a better place for discussions. I just wanted to ask if i got the idea right, so thanks for your answer.


...これは？？


@Like-all no need to dick around here. Consider reading the NEWS file from 229, I think it's very clear that we main compatibility.


gfgfg


これ前のコミットではこのまんまだったから、直すなら直してからコミットしてね


a mostly unrelated change; just filling in this missing documentation while I'm here


fff


Mantas Pipiras


I would also prefer supporting older versions of OpenGL. However, the author of the PR had segfaults when enabling it. See https://github.com/aochagavia/rocket/pull/11#issuecomment-182109688 for more information.

Feel free to open a PR if you are able to fix it. It would be nice to support as much platforms as possible.


uh oh, are you sure about === ?


@twangboy I like it. Thanks.
I am not quite getting the point about some version mismatch? I woudl like to understand, so I can bring the old salt-winrepo up to parity and chnage it, if it is currently leading to mismatched versions somewhere? Or did I misunderstand you about what the initila problem was that led you too look at this one?


@poettering So? What does it mean "previously existing timeout settings"? How do you detect them?


I need you to agree with the licensing changes in #102 before merging, is that ok with you?


status should be part of a state. This way it'll work, but changing status won't trigger rerendering the app.


@muaddieb Please push new version of the model for review.


mr.new v1


Not sure, maybe someone will look at it and decide to fix it.


I tweaked the formatting a bit.


Done.


The dictionary definition here is not PEP8; the colon needs to immediately follow the key. Check out this alternate definition that also removes an unneeded variable:

```python
@given('a font having {color} highlighting')
def given_a_font_having_color_highlighting(context, color):
    paragraph_index = {
        'no':           0,
        'yellow':       1,
        'bright green': 2,
    }[color]
    document = Document(test_docx('txt-font-highlight-color'))
    context.font = document.paragraphs[paragraph_index].runs[0].font
```

A couple other bits:

* I reduced the number of members since we only need three now.
* I changed the wording to "a font having {color} highlighting" because it reads naturally in all cases after changing "None" to "no"
* I reduced "highlight_color" to "color" in the parameter name, again reading more naturally.
* I changed color_index to paragraph_index, removing the need for a comment
* I removed the leading cap on the color names (and "no") because during the test they appear as a word in the middle of a sentence and are not proper nouns.

Some of these are arguably stylistic changes, but you'll see them consistent with conventions in the other tests (at least the more recent ones :)

On the last line, you'll want to browse the XML of the document to make sure the run you're getting is the one you want. Word breaks runs at odd places during editing, and it can lead to surprises. The main one we'd be worried about here would be a false positive, which might be unlikely now that I think about it, but just so you know to be circumspect when using runs on UI-mad documents. Often I'll make the paragraph all in one shot so it goes into a single run and not edit it afterward. But browsing with opc-diag is what I'll do before committing, patching the XML by hand if necessary.


Should this be 1.0f instead of 1.f?


How often does that really happen? ;)


This conflicts with line 701:

```yaml
- match: '^(?:\s*$|(?=[ ]{,3}>.))|(?=[ \t]*\n)(?<=^===|^====|=====|^---|^----|-----)[ \t]*\n|(?=^#)'
```

If you add a line separator to your markdown file:

```md
---
```

The whole highlighting is messed up after that line.


@natif I would go with a different class name because in case you change the padding then the name won't make sense.
Maybe you can use something like `.small-padding`


a tool that might be handy the package to have: http://ifacethoughts.net/2009/04/06/extending-bash-auto-completion/


I was going to build on all branches, but tag them in docker (with the branch name?) so that only master/release are installed (once that's automatic).


@Creeparoo Please always add informative commit messages ☺!


#23652 Adds a failing test... good eye, @egilburg


You need to create the enumeration and import it here (and include it as part of this commit). The acceptance test won't run without the enumeration and if the acceptance test won't run (because of an import error or undefined name) then it's not done yet.


This test ensure #17789 is fixed as well.


TimeoutStopSec= is a previously existing one, because it existed before v229. Hence for it we interpret both "0" and "infinity" as an equivalent way to turn off the timeout.


Oh right, I forgot about tags... That should be quite straight forward.


@JustAkan 그렇군요 확인해보겠습니다


I think that's as far as I can go on this one until you get the enumeration defined and in there. Let me know when you have it and I'll carry on.

Btw, your first commit is live on the new feature/highlight branch on the main repo here:
https://github.com/python-openxml/python-docx/commits/feature/highlight

Congratulations! The first of many more I'm sure :)


@jreback Shouldn't it be enough to add dependencies on `numpy x.x` and `python x.x` to get these kinds of strings? Or did you do that specifically to get the `{{ GIT_BUILD_STR }}` in there?


I think this happens when moving from any server if the VM was created with a new version of libvirt fwiw. Old versions didn't embed specific CPU info but newer ones seem to.


@jorgenevil håper du ikke har stylet css på denne id'en? forandra navnet, siden dette er public code.. ;)


There also might be a benefit to having these sections (e.g. maybe more CPU features like AES-NI could be available inside the VMs), not sure.


I did consider using  Function1. Since Function1 extends j.u.f.Function ideally derive4j would use Function when in contra-variant position and Function1 in co-variant position (return type). But currently we must make a uniform choice and will trust you that using Function1 over Function offer better user experience.

For Supplier/Function0 I think Supplier is never used returned from any generated method so I don't see any advantage on using Function0 over Supplier (since Function0 extends Supplier).


This broke lightmaps on Mac/Intel HD 3000


Numbers do not make sense, an aluminium ingot has a weight of 2 kg while a aluminium can weighs only  0.01 kg and aluminium foil weighs 0.8 kg per 200 foil. This would make aluminium appear out of thin air.


Plant it. I have on field of it. The seeds in some harvest craft garden.


Well, I did spend some time looking into it: the problem is that `parse . display == id` does not hold in general for the `VersionInterval` type. Maybe one can come up with a "normal form", so something like `parse . display . normalise == normalise` would hold.


極小MP3モジュール [M2801002]　http://www.aitendo.com/product/11667
は、抵抗値で動作モードを決定するADKEYという機能を使用していますが、
そのADKEY端子にArduinoのPWM出力にLPF通した信号を接続して、
制御してみました。
Arduino IDE 1.6.5 for Mac にて、作成しました。


Added a test.


Note: Dispelling a buff _from a friendly_. This should be obvious, really, but I forgot to specify.


</html>


please modify b-5 to b-5.1


Yours was reverted?


@Ark-kun You should use the CNTKTextReader which is coming soon. In the meantime the SparsePCReader is probably your best choice.


añadido archivos meta al .gitignore


Renombradas


added python requirement


@2franix Well done! Thank you.


Hi @kd7tck!

Yes, it's almost ready, I set a deadline for Monday 22 of February. On Wednesday 24th I'm doing a pitch about raylib and It should be ready.

I've been looking for some big bugs around all the TODOs spread in the code.

Right now I'm focused on one issue I want to finish for 1.4: Raspberry Pi input system. Yesterday I was reviewing the RPI input system, I re-analized the decisions I took and I was investigating how to improve it.


I think we can reasonably say that you (as maintainer) are an exception :). Either way: If you decide to keep the commented code, maybe your explanation of how it could be fixed should be added to the code? (Kudos, btw, for looking into it.)

Review comments in GitHub have a tendency to be lost post-merge :/.


I am not sure why this condition was added in the first place, for the record. Git blame shows it was added as part of the last dispel system rewrite by @Kandera 4 years ago in 01bc6c483bc0756850ee8853f98bd8de4a2316bc, with no particular remarks about the nature of this check, so I am going ahead and assuming misguided efficiency.


Nice work!
Isn't it path for USB? I'm using bullhead.

/devices/soc.0/f9200000.ssusb/f9200000.dwc3/xhci-hcd.0.auto/usb*


commented out code is bad for the project. Let's please get rid of it.


heh, good catch, totally forgot about this, I just pushed this


Shouldn't we add this as `ModelType.VOID` instead?


We should eventually use $TRAVIS_REPO_SLUG for the actual container name, which would make it easy to use individual forks...


Why use __ in save-dev?


原文は"before 8.1"なので、訳としては「8.1より前」が正しいと思います。


lol


There's actually already a link in the comments.


変更部分について確認しました。


@egilburg I think you're right, it appears it matters in this case... added a test case that fails here: #23652


Consider using Interfaces instead of concrete implementations.
**List<LightWeapon> weapon = new ArrayList<>();**


prova merge


revert this.


Nice. Just ran into this too.


Synced this revision to basewsw race.


hey @nivla, can you tell me what is the date of publishing cycle.js fundamentals at egghead.io? I can find it on the place.

thanks


Sorry .confusion lol.
Will revert revert lol
Until then you can try it


Apologies, I missed that because of the "diff context" limit.


so you don't really "change" the documentation, right? You rather remove it. So I think it'd be nice to change the shortlog of your commit message to `tests/bearlib: Remove doubled documentation` or so (also notice the space after the colon but not before, this always confuses people but it's just as with normal english ;))

Otherwise looks good!


Thanks for this Steve - do you think I should release a new version?


@poettering maybe this is ok for you. But I thought that systemd was started to make complex things simple. So, now you introduce 2(two) different ways of doing the same for related actions. Simple things became complex. Now we have to memorize which settings were introduced before v229 and which settings were introduced after. With "0" as "never" and "-1" as "no" these things would be clear, more intuitive and obvious. And, of course, there would be no need in "backward compatibility" layer. I just want to keep things simple. Do you?


Hi,

I committed your deprecated methods work into the main codebase here:
https://github.com/openemr/openemr/commit/82f09432d4aa5a3e5a65ba364a1a7893d04d02a1

Note I made several changes to your code, though.
Made two minor fixes here:
https://github.com/bradymiller/openemr/commit/524afdae8b00bc9e2307b93a2df60872b637af2f
And then removed the html2pdf changes here(code was causing a fatal php error):
https://github.com/bradymiller/openemr/commit/732a43dc1783ef2acdf7ee234edaebc83c095d67

My suggestion on the html2pdf is to possibly try to consider upgrading it to a more recent version(http://www.open-emr.org/wiki/index.php/Html2pdf) versus keep trying to get it to work. The code can be tested on OpenEMR when do following in OpenEMR:
Patient Summary Screen->Report->Download PDF

-brady


The main purpose was to make practice with design patterns and algorithms.


ack, did you test it btw.?


ack


Good Job


isn't it 0.4.1.dev?


unack, looks good besides that


Lehet, hogy a geoclue-dev mégis kellene a webkitgtk2-dev -be?

Majd kiderül.


I am sorry, but I like my bikesheds yellow.


Who made the loader?


Deleted CNAME file


does this have any other side effects?


IIRC this may have some effect on the test execution order? Also we need to add an argument so it actually tries to execute it in parallel by default, right?


@poettering Well, the one thing that became clear in this negotiaton is that you are an asshole, Lennart. Keep wanking on your bikesheds. Good Bye.


ok this looks like dark magic, pytest looks for this file and runs that function on it's own?


please b-6 to b-6-1


please b-7 to b-7-1


maybe add a check that the output is only the one line and not some json?


OK, I was just overthinking it and wanted to double check since you're accessing the items of that array in the normalizer function. I'll have a PR shortly.


just do `# Ignore PyLintBear` instead :)


unack


hey maybe this test could well be a documentation example.

```
>>> from ... import execute_coala
>>> from ... import VERSION
>>> retval, output = execute_coala(main, 'coala-json', '-v')
>>> retval
0
>>> assert output == VERSION
```


:wink:
`"Error running ['zfs', 'create', '-V', '3G', '-o', 'mountpoint=/install/swap', 'antergos_sijp/swap']: cannot create 'antergos_sijp/swap': 'mountpoint' does not apply to datasets of this type"`


readme 文件


Can you not use these vars in the `packages` section?


THANK YOU!!!


this mapping looks very... useless


unack either you do it or not, you shouldn't take a chance of getting this wrong


`test_netrc_present` fails, `_NETRC_MACHINE` must be added.


unack, don't use `action='store_true'` please since it'll always store a value as it has a default which will store a useless default in the users config if invopked with `-s`


our tests catch that for coala but not coala-json :/


just use `log_printer = None if args.text_logs else ListLogPrinter()`


no need for this variable, it doesn't help much


just use args.text_logs directly


@wsnipex: Can we use the `TARGET_FOO` defines on linux et all when building binary addons standalone (i.e. not through Kodi depends)?


Fixes #1


@mphprogrammer: This was a very minor fix.


Nice work... XD


This variable (and the `y` one below) is unused, should it be simply removed to remove gcc warnings about this or was it supposed to be used and should the code doing it be added?


Nice work!
I didn't use OTG yet. But isn't it path for USB? I'm using bullhead.

/devices/soc.0/f9200000.ssusb/f9200000.dwc3/xhci-hcd.0.auto/usb*


No problem. Which ddl problems did you recieve, I'll try to look into it? Will this get remerged if I add a parameter for this?


Indeed, thanks!


this line should be deleted


Thanks!!! Really nice!!!


Done.


Initially appointed as a revizor minifikator CSS files, and other files minification (.js, .html, etc) are a side effect of this process.

But if you want to compress other files not related to CSS, then it is necessary dleat a more flexible option, for example to do a search on the same mask, instead of adding each selector in an array.

Example:
nonCssFileSelectors: [
'js--', // for .js--submit-button (replace in getElementByClassName("js--submit-button") and $('.js--submit-button'))
'id--', // for #id--submit-button (replace in getElementById("id--submit-button") and $('#id--submit-button'))
]

But it is better to implement it even easier to make non CssFileSelectors: boolean and search selectors in not CSS files by namePrefix, while the selectors have to write example .js-submit-button__, #submit-button__ and all.


commit comment


<!DOCTYPE html>

<meta charset="ISO-8859-1">
<title>Insert title here</title>

  Dictate


The commit message talks about `dhcp-compat` yet removes installation of `dhcp` one. Am I missing something?


hello sir...
i want to learn kernel module programming but i am not getting how to start with this topic...can u please help me to start with this...if u can...


no u have to construct the build string explicitly (if u want it)


<!DOCTYPE html>

<meta charset="ISO-8859-1">
<title>Insert title here</title>

  Start your Dictation


他にも同じ間違いがたくさんありそうですね。気にしないことにしていたのですが、なるべく、修正のついでに直すことにします。


테스트


Yep


See https://twitter.com/silv3rwind/status/698468639163465728 for background.


Awesome. :+1:


Yeah, I think it's  worth it as it encourages 'automatic' upgrade to a
version that def will work if someone zaps the pinned version of
semantic_version
On 13 Feb 2016 11:11 a.m., "Hugo Rodger-Brown" <notifications@github.com>
wrote:

> Thanks for this Steve - do you think I should release a new version?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stevejalim/django-package-monitor/commit/6ab2996f758411482855ec391d7cecfee2226cbb#commitcomment-16077725>
> .
>


Это хоть и генератор, но не очень-то ленивый.


И не очень-то правильный, даже если так


А, не, отбой. Я упоролся


Once a block of data is written in google cloud storage it is immutable. Failing the entire upload if the digest and uploaded data get out of sync isn't great, as it would force the client to restart from zero.

There might be a way to prevent computing an incorrect digest or at least detect when the client uploads a different chunk on re-try. You could keep two digest states:
1. one that is guaranteed to be for an offset < current file size
2. one that is updated while bytes are written to the FileWriter (ie has an offset >= current file size)
If all is well, after calling Close the offset of the second digest state will be exactly the size of the file.
In case of a problem, we can tell the client to restart at the offset of digest 1. Once we reach the offset of digest 2 while receiving the retried chunk we can check whether the digest matches digest 2. It this is not the case the client tried to upload different contents.


Resolvido


Yes, ok


reack


You use the quoted and escaped name here, but not for example in line 33. Also: Lots of line breaks


Why not oreditc pams wax with the others??


It is working a bit buggy ATM. Any Suggestions?


Waar gebruik je deze kolommen voor? Ze lijken nergens voor nodig.


I'm pretty sure you don't need to do it (apart form the `git_built_string`part. E.g. this recipe:
https://github.com/matplotlib/matplotlib/blob/master/ci/conda_recipe/meta.yaml
results in these packages:
https://ci.appveyor.com/project/mdboom/matplotlib/build/1.0.817/job/vskn5wo2r76g0j0q/artifacts


Okay, thanks.


Nice!

(Again, I don't necessarily agree with the particulars, but it's good to be consistent.)


I think `\b` should be interpreted as just `b` because `\` before non-special character has no meaning.


Ok, I'll take another peak at it.. but I must admit I'm intrigued by your PigOSC project.
(did you miss to attach the files..? couldn't find them)


ack


This hunk should be reverted. (For example, `\\` should be interpreted as `\\`)


An elegant solution. However, can you guarantee that this way a player may not enter the instance map in a ghost form? I.e. side effects seems to me less predictable than in my version.
Anyway, either this or (my) PR 78 should be choosen.


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


Thanks all! since my images were in the image file with all the other items
I was thinking I was all set. But, I will fix. before or at class :) I have
a breakfast meeting before class.

See you in a few!
Sandy

On Fri, Feb 12, 2016 at 9:47 PM, Matt Bruton <notifications@github.com>
wrote:

> Made a couple comments on the code, could resolve some issues with your
> images not loading correctly.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/abidina/pastOurTime/commit/5f01a9a4b20a79f31f33267b5b2a9d3d72e5dc3c#commitcomment-16076379>
> .
>


AJ, check output for eagle compile. Same change can go in there I presume ;-)


It doesnt seem to have any other side effect
pytest-xdist does change the test execution order - but right now there is no fix for that https://github.com/pytest-dev/pytest-xdist/issues/46


Hello,
if you please can you tell about the Android App who compatible with GPS-Manager
thanks


Oh, ive not added `-n 4` to the args because the output looks ugly and it doesnt show info properly.
Check the issue I mentioned above.
Ill do it after pytest-sugar supports xdist


Code dupliqué en trois fois il faudrait un template twig générique pour la signature


`conftest.py` is the name of the configuration file of pytest

The `pytest_unconfigure` is one of the hook which pytests exposes which Im latching on to here


I can do too.


I think this should be a uint64, not a string


and `pass` inside the function ? Seems counter intuitive to me


On 13/02/16 02:31, Jimmy Yih wrote:
> This commit might have broke multinode gpinitsystem executions.  After closer observation, the for loop at line 1358 may be looping over segments that can be on different hosts?
>
> The QE_PRIMARY_ARRAY variable that is being looped over is initialized as:
> ```QE_PRIMARY_ARRAY=(`${EXPORT_LIB_PATH};env PGOPTIONS="-c gp_session_role=utility" $PSQL -q -p $MASTER_PORT -d "$QD_DBNAME" -A -t -c"select a.hostname, a.datadir, a.port, b.valid, b.isprimary from $CONFIG_TABLE a, $GP_PG_VIEW b where a.dbid=b.dbid and a.content<>-1 and  a.p\
> referred_role = 'p' order by a.content;" 2>/dev/null`) > /dev/null 2>&1```

Oh, bummer. I didn't realize that SET_VAR changes all those variables.

Will revert, thanks!

- Heikki


And this should probably be `q.String("ip")` ?


no function at all, the doctest goes directly into the documentation comment of the main function


ack


I think we need to keep `abs(theta)` and `abs(r)`, because vertical lines in Hough parametrization can be either `theta = pi/2 and r > 0` or `theta= -pi/2 and r < 0`.
By taking the absolute value for both, we ensure that we get the parametrization where `theta = pi/2 and r > 0`.
This is important for us to make sure that the edges are then sorted from top to bottom and left to right, so that eventually `blindboard_matrix[0][0]` will be the upper-left square on the chessboard


@sils1297 It works if you install coala normally. But the readon i mention `try` is because i wasnt sure about the windows - but now that I have tests, i can check that out :)


为什么在3.10版本中要把Sprite3DMaterial类加入忽略中不导出？我之前的lua代码用到了这个类，替代方案是什么呢？


i dont merged


General note: We should have the keys as constants somewhere


and dont work


:bow:


reack


OK. Will do...


Hi,

Your Zend library upgrade tested well, so I brought this into the codebase here:
https://github.com/openemr/openemr/commit/8b4faf94210dc68d562539a18fd0fda12080d0fb

-brady


Agreed, we especially need to look again at aside. The description isn't per se wrong, but the way it's used in the examples is not really in line with HTML's idea of aside. It's also going to cause issue if we start using main tag/role on pages as asides are sortof assumed NOT to be children of those.


Aha ! alright - good idea


Benutz doch bitte mocking oder stubs, wenn du den ReplayLogSaveService testen willst. Wenn sonst irgendwas am filesystem kaputt ist, wird der test auch eventuell fehlschlagen, obwohl filesystem funktioniert.


Done


aber ich brauche ja kein fake hier, ich brauche wirklich die Folders wo ich die Daten reinspeichern kann, kann ich dass so einfach mit Mocks machen?


@hakanersu merhaba, Listeye `"call_record": "Yok",` eklenebilir mi? Stream etme özelliği API'ye eklendi. Ses dosyalarını kayıtlıysa direk uuid ile dinleyebiliriz.

* https://github.com/bulutfon/documents/blob/master/API/endpoints/call-records.md#urller


Das geht bestimmt. Wenn du z.B. den Zeitpunkt den du loggst nicht selbst generieren würdest, sondern ihn aus den Truffles/Commands holen würdest, dann könntest du einfach mocken. Mit when(myClass.myMethod(myParam)).thenReturn(returnValue)


@Niels-NTG, please NEVER update any dependency manually! Everything has to be resolved in bower.json, see https://github.com/generative-design/Code-Package-p5.js#developer-setup.


请学会像其他同学一样建文件夹和文件名。以后代码多了好管理和交流。


@onurozgurozkan Ekliyorum.  Steam ozelliginin eklenmesi guzel olmus.


Kaynak : http://api.bulutfon.com/docs#!/Call_Record/get_call_records_uuid_stream


Das ist ja auch so vorgesehen, nur Commands haben das noch nicht eingebaut.

Aber was du vorschlägst ist glaube ich irrelevant, weil alles was FileSystem macht hier, ist mir den Pfad zu dem replaylog folder zu geben (also ./data/replaylog) und nicht wo genau die sich dann befinden. Das wird alles bei mir intern gemacht


Sorry for writing here. I have to ask you something, how can I contact you?
If you want, you can contact me on the e-mail written on my GitHub Profile.
Still sorry,
Slim


Well the entries are used in a structure defined in eagle's gpiomux but there is no function using it so technically both entries and struct can be removed. I'll compile and see, Erik


`chihaya.Scrape` does not have a field named `Downloaded`. This does not compile, but we never include the package, so we don't notice.


Hello, where is now this option? ;)


It resolves #26


oups ! thanks


Honestly I would prefer to keep the comment but add a new one to say that game is working now. If someone change the code in the future, he knows that D2 can help to find issue.


@hishamhm deps.show_dep and deps.show_version are not used outside deps module and `internal` argument is not used anywhere. Are these functions public interface in some way?


Spurious whitespace changes.


You mean doing `- ghc-$GHCVER` instead? Probably we can. Could you test this?


containername looks wrong as well (one l, test)


Sure.


Have you read the section about **Placeholders**?


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 259](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=782&buildTypeId=XbimEssentials_XbimEssentials) is now running


removed regex1.json


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 259](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=782&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:01:30


This is likely a good time grab the most recent development branch on github or sourceforge and work from that.


Bad @mikeb01 :-) This commit had a lot of build failures.


lol nice. But what if we change the actual email to `callmeAThere`? :package:

Just kidding.


I recommend using `***` since `---` is reserved as a yfm (yaml front matter) delimiter.


>Also best to reuse our own API methods: audio.on() audio.off() audio.saveSetting(), etc.

I had planned to do that and then realised none them deal with only EEPROM alone. I strictly wanted to change the value in EEPROM, then let the existing init code do whatever it did before, further on.

*on()* and *off()* actually enable or disable the sound, which could possibly generate some extra code (now or in the future) which wouldn't have been created if they weren't called by the sketch. *saveOnOff()* saves the current enable state, so *on()* or *off()* would have to be called first.

If we want to add audio API functions that *only* affect the value of EEPROM location *EEPROM_AUDIO_ON_OFF*, then these new functions could be used.


Can you squash this together with 046fd47?


I've added the additional l in the latest commit. Sorry for the new repo it created on your account. We'll have to switch to something like $TRAVIS_REPO_SLUG or similar to actually push to the real docker repo, since there is no user with the username mozillach on docker hub.


Can you wrap this in #if HAVE_ARPA_INET_H?


These casts seem unneeded?


This is great, thank you! I wish more people contributed like this.

I'm going to merge, but then make one small tweak myself (just to avoid going back and forth on it). You can simplify it slightly by putting `master = False` on `RemoteUrlCommand`, rather than on both `CopyRemoteUrlCommand` and `OpenRemoteUrlCommand`.


I have the yaml definition commented out, so it doesn't bother me, but I think this language definition may look broken for a lot of people, because using `---` is perfectly valid markdown syntax.


This commit contains changes by @rohitacc


As I understand  on openpli does not exist /etc/image-version.
I think that for imageversion should be used /etc/issue


C99 comments are ok, see https://collectd.org/wiki/index.php/Coding_style


Est-ce vraiment une bonne idée d'alourdir ainsi la loi? L' "illectronisme", qui existe sans doute (surtout chez les anciens présidents, amateurs de mulots), présente une multitude de variantes & de dimensions : on peut savoir envoyer un mail ou/et visionner YouTube, sans toutefois se débrouiller dans la recherche - et la hiérarchisation - de l'information.

Or, une telle compétence va au-delà d'Internet (d'ailleurs, la pré-éminence de l'image et de Twitter sur Internet montre qu'il y a là une part... d'illettrisme, ou du moins, d'une désaffectation envers l'écrit, que tout prof de Terminale constate quotidiennement).

De même, rédiger un CV n'a rien d'une "compétence spécifiquement numérique" (sauf à dire que l'usage de Word est la même chose que l'usage d'Internet).

Enfin, on pourrait aussi taxer d'illectronisme le fait d'être incapable de coder ou même d'utiliser une syntaxe style wiki ou blog (html), ce qui est le cas de bcp de gens (et pourtant, mis à part que cela est certainement souhaitable d'un point de vue démocratique, il n'est pas nécessaire de savoir coder pour utiliser un ordinateur).

Bref, alors que l'illettrisme est un phénomène relativement bien cerné (au moins théoriquement,  à défaut d'en connaître la véritable ampleur sociologique - considérable, il suffit de voir à quel point les textes à l'épreuve de commentaire en philosophie au Bac ont diminué de taille ces dernières années), l' "illettrisme numérique" semble trop polysémique et variable pour pouvoir faire l'objet d'un tel énoncé dans la loi, réduite à l'état de déclaration d'intention.


Heh, arrow functions can be confusing sometimes :P


I think it's more correct to yield here for falsey values and potentially raise an error instead of blankly not rendering something. We have checks that objects in the collection must respond to `to_partial_path` which neither nil or false does. Thus this should raise here: https://github.com/rails/rails/blob/e183227e5c7665b6505dd319724f1a540946b7ce/actionview/lib/action_view/renderer/partial_renderer.rb#L487


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/tests/processes/ProcessingTest.py`, line 300.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/tests/processes/ProcessingTest.py
+++ b/coalib/tests/processes/ProcessingTest.py
@@ -297,7 +297,7 @@
         self.assertEqual(len(file_dict), 1)
         self.assertEqual(type(file_dict[self.testcode_c_path]),
                          tuple,
-                        msg= "files in file_dict should not be editable")
+                         msg="files in file_dict should not be editable")

 class ProcessingTest_GetDefaultActions(unittest.TestCase):
```


결국 저번에 한거랑 똑같은거 하는데 공부한 걸 요약을 해서 가져가야 함.


Proposition d'amendement à cet amendement: "cet article [de loi] s'auto-détruira à la remise du rapport".


We actually prefer return (1) above return 1


i didn't know the exatcly date but the author shared the link in twitter on january 22.


fyi, this change because experiment is not a string, but an experiment object


and this change so we only need to clean once.  There are more similar changes below.


Tabs here please.


What is this please? 😭


Add a octet string transfer with output only to avoid destroying the source buffer with the data returned from the slave device.
Also some simplifications to the code


Couldn't path completions be provided by an atom package (I use [this one](https://atom.io/packages/autocomplete-paths))


unack would you mind adding some explanatory text :) this is documentation after all


This should be below time_offset.


AddPerformanceTestForOctree (DeadParrot) - 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)


a for can have an else?


Im not sure what sort of documentation you have in mind.
Should I document what the `main()` function does or what the testing is for - I find the doctest is a little out of place - it shouldnt belong to the documentation here


current_page? current_user ?  'class="active"' : ''


Indentation is off here. In general we prefer 2 or 4 space indentation for new files (https://collectd.org/wiki/index.php/Coding_style)


Yes it can - one of the things i like about python !


at least explain that htis is the main json entry point and how it behaves, if it fits, use this doctest as an example if it doesn't move it back to the tests.


I call it code re-use


A priori on ne peut que se féliciter de tout renforcement de la CNIL.


is it stable on Note 4 (Exynos) ? i ll install to my friend's note 4


@JustAkan 작업하다가 이 부분에서 막히네요
https://github.com/multirom-nexus6p/android_kernel_huawei_angler/commit/49f431b230de7fedc3a5a8b01001ebe614c38030#diff-287d2c62df40ba9d834f5994a084d060

arch/arm64/kernel/relocate_kernel.S 소스의 45, 137 번줄을 보시면 하드 코딩 된듯한 주소들이 보입니다.
본래 작업하시던 분의 소스에는 이런 부분이 없어서 난해하네요.


This is not the correct place for this piece of code, as it will not have the intended effect

You must wait that all the ei,attackedBy[Us][ALL_PIECES] had been calculated,
and this is outside of evaluate_pieces.

For example, in evaluate_threat  would be fine.


what is a ddf file?


Šlo by tohle dát to konfigurace pluginu a pak to PHPčkem jen předat do JS?


I was calling the temperature series as ``Tramp`` rather than ``Tseries``, which is too close to ``tseries`` and will cause confusion.  However, this change almost certainly breaks something so rather extensive testing should be done to make sure that everything works before we deploy.

Quick question...do the temperature ramp scripts work with the version of the software at XPD?  I am a little worried that they maybe don't seeing this 'Tseries' in here


> On 4 Feb 2016, at 8:18 am, Robert Xiu <notifications@github.com> wrote:
>
> its fine, Just make a requirements.md saying the requirements of this project.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/Lightning-SF/PythonDictionary/commit/eef1cb3cbfb1386e83a943d742ed1a4b46c521f9#commitcomment-15873592>.
>

Uh robert i just saw this and um is it even meant for me


Typo :-)


We should probably also `OpenPeerStore` and `OpenClientStore`


to resolve locked up


I didn't check, but what happens if the socket is udp?


Ok thanks


https://github.com/multirom-nexus6p/android_kernel_huawei_angler/commit/47a102a4fe076281cc181ca7f47539ee275023b0
추가커밋도있으니참고바라시며.
아마도 넥서스에맞추기위한주소아닐까요?
일단은 넥서스6p를우선으로따라가보는게좋다고봅니다


like I stated in my comment I'm not really the c guy ;) so I tried to figured the changes for python 3 from the python docs. but as far as I see the changes seem okay. I will get some of the other guys to check it too and if they are fine I will pull it


@trevnorris The `TryCatch` is needed in workers to distinguish between `HasTerminated()` and `HasCaught()`. `IsEmpty()` will be true for both cases but terminating workers is not a fatal error so we need to distinguish between these cases.


@bnoordhuis


We need to import at least some servers/middlewares (maybe not in cmd/chihaya/main.go, but cmd/chihaya-basic/main.go), so they get built/tested


Kuba, nie możesz w tym projekcie używać warstwy dostępu do danych. Wszystko musi być opakowane serwisem.
Powinieneś dodać nową metodę w webservice.


seems to create a problem with the regex on the buildserver?


Thank you for this idea. I will add a message separately for each flag. I will try to keep this in mind in future pull requests.


Now Animation dock is between side docks and doesn't take space for Navigation dock.

![image](https://cloud.githubusercontent.com/assets/11696990/13027797/77afb9da-d25d-11e5-87aa-724f523b4e27.png)


state.applicationReducer is no longer updated


This should be `mob.stop_animation` (JRuby will convert it to `mob.stopAnimation` for us).


I think that all basic features are working. Minor bugs like robotic call sound and too loud speaker volume will be fixed in next build (located on BasketBuild).


Why do we create/delete by clientID, but find by peerID?


There's nothing to fix here. It compiles and works just fine on an OpenGL 2.0 system with these [specs](https://gist.github.com/petevine/74f8a09fbd1238255a13)

Any problems the OP had with his patch should have been reported upstream.


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 147](https://teamcity.labkey.org/viewLog.html?buildId=303111&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 8 (1 new), passed: 42 Build time: 01:24:29
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilters: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyLogGutterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyDensePlotBrushing: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisFilter: <no details avaliable>

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

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

```


Removed superfluous 'Log' header.


AddPerformanceTestForOctree (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (984 of 984 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-28.54%25-green.svg)


Great. :clap:


# Hello Swift, Hello World


We can `RLock` this


thanks i ll test it soon


@ashtonthomas you better listen to your reviewers bro


I will experiment by checking if the env is a worker thread and whether it's been terminated in the .IsEmpty() paths


Changed a few sentences etc in Q1.


Ah for the rebase


Jep. I guess the older CMake version tries to interpret this as a regexp. Did not do this in CMake 3.x but I now replace "+" by "X" which should fix this


isn't this the same issue?


I wouldnt change it that way because its way to general in my opinion.  At least limit it to sg and sd like

elif _have_linux_sgio and  d[:7] in ('/dev/sg','/dev/sd'):


Eh @w5860363 is the author, but apparently using his displayed email has assigned it to someone else


А что именно не понял?..


This commit should be squashed up


AddPerformanceTestForOctree (DeadParrot) - x86_64-MacOS-10.9-clang: OK (2080 of 2080 tests passed, 3 test warnings)

 * 3 tests had: AUD 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)


Вот это — ок, отдельный корректный тест.


This thread has gotten out of hand. I am making a motion to establish a code of conduct around the proper and respectful way to discuss matters pertaining to the superiority or the promotion of false beliefs toward carnivoran species and how such comments and beliefs should be received by and reflected in the masterful project of the `pet-store` while taking into special consideration the long-term impact of such decisions on the growth and prosperity of community dependent upon this project, the `pet-store`.


isMuchSmallerThan() :joy:


we should propose to std library to add the  std::lo_fa_solo_per_rompere_le_palle()


I think this is still needed. If no output is sent to the screen for more than 10 minutes Travis kills the job.


ack


But the bug is not solved then, only good managed ...


yeah that looks like a reasonable addition :)  if we get all the other stuff with the c module sorted I will pull this


Э! А зачем мы отдельно писали и тестировали метод tmdb_movie_by_id, если здесь его не используем?..


Looks like this got modified after a rebase?


We could just use the four last bytes of the infohash for this, instead of the FNV hash


Same here


@mr-sneezy - you might be interested in this alternative. It doesnt have any of your new changes, but check if this functions properly without inducing a delay in the bluetooth -> ecu communications


@joshwwarren Here's my solution to the problem.  I put it on Github because it's easier to read here.

I looked at the other people who completed it, and it seems I had a different approach.  Mine is more of a brute force method, I guess.  I knew the answer would always be found in an array of consecutive integers, so I build a list of possible ranges that could be considered given the inputs.  Then all I had to do is loop through them all, find one range with a sum that matches the total, and then return the proper element.

This is what I was talking about, not knowing what a solution will turn out to be until you get enough test cases through it.  I just kept pushing more and more test cases, each with hardcodes... until I saw a pattern.  Then I just had to refactor.


Did you deliberately remove the 014 error? I think this occurs when you have a US account and you are trying to send more than your monthly limit (60k or something like that).


@jkroepke PHP 5.3 is dead, why do you still want to support it?


*feeling like giving my opinion* I prefer the uncolored studio title


didnt investigate why the gp tells you that some gradients are 1e200, i only managed to not crash the program, in demo view it is more important, after the deadline we can try to solve it, imo.

Besides i think is better if i waste my weekend on the implementation of the multibrach explorer with probability atlas


still a space before the "Remove" missing


Quoique l'exposé des motifs soulève des points intéressants (ex : bots de messagerie), le renversement de la règle & de l'exception, ajouté à la clause du "consentement" (souvent extorqué, en raison de l'impossibilité d'usage associée au refus de consentement: cf. ce qui se passe pour les cookies), conduit à vider l'article de son contenu.

De +, la suppression de la notion d' "atteinte au secret des correspondances" est grave, en particulier parce qu'elle permet d'arrimer cet article à la vie privée, ce qui comporte quelques conséquences au niveau du Conseil constitutionnel et de la CEDH.


I'm not sure how maps are implemented, but I guess we will slowly fill up memory doing this, because the pair is still kept in the map. I think we should `delete(shard.peers,key)` this


Not sure 'sup with --author
![capture](https://cloud.githubusercontent.com/assets/2879479/13027944/6eee6c3a-d247-11e5-947c-3310c53c7c5a.JPG)

GH however still resolves it wrongly


Totally agree


The HTTP API uses the legacy response decoder and returns the error code in array key "error" (Decoder.php line 103). So the `$entry` variable should contain the `'code'` as a key and not `'errorCode'`. Where does the `'errorCode'` key come from?


it looks like the new commits are fetched but not merged, which is odd, if that's a pull.
I documented that here: https://github.com/supercollider/supercollider/issues/1864


(swicthing over since that's not about that commit anymore)


Now I'm confused. If a custom index.html is a one-off page, why would it have variables in it?

Okay, that's cool, we can be consistent and have variables in the custom index.html.

I don't understand why you changed this `throw` though. Why not just have a custom index.html page (with `butterfly-pages` in it) but with no `butterfly-content` tag in it?


Yeah it works!


Inline this, remove `peersToDelete` ?


From the original letter

>Issues often accumulate content-less “+1” comments which serve only to spam the maintainers and any others subscribed to the issue. These +1s serve a valuable function in letting maintainers know how widespread an issue is, but their drawbacks are too great. We’d like issues to gain a first-class voting system, and for content-less comments like “+1” or “:+1:” or “me too” to trigger a warning and instructions on how to use the voting mechanism.

Why this is just dripping with irony ...


thanks, I'm working on it using redux ui-router middleware


Person constructs require a name. Besides, I don't think robots harvest RSS for spam (and spam filters are pretty good these days).


@sils1297 Okk


sure, as long as they are standard preprocessor defines, not something we invented.


@Nyeriah The mailbox has been written off. It doesn't matter


That's a good use-case for sorting and I like your implementation. I think your change is good (default is `-1` unless specified otherwise). I'll integrate this.


Why do this, instead of just `/channelname`, it's ugly af.


add home work


I have a question: Why can't we use the new function for both needs (and get rid of `getSpellCooldownSmooth()`) for instance?


BGMの追加 #152


Looks sf thanks to the architecture you put in place. :)


Looks pretty much like the commit before. ;)


@Robotia Where are the files for    net/openhft/affinity/AffinityLock?


@RobotXLabs @nywillb was the person who found it on <http://cssload.net>


Nice.


Excellent :)


your choice, all I wanted was to remove email.


I'm not entirely sure I understand. It seems like you want to use the filename to specify the URL (which makes sense -- SEO, etc.) and specify the page order, and use meta-title for the actual displayed title.

Let's assume I implement page sorting (which I plan to). Is it enough to let the user specify the filename, and a `meta-title` tag that directly controls what title renders in `h2` tags? Or do we need something more?


I made a note to extract the `class="blog-post-title"` part into a butterfly HTML attribute or in config.


Ok


It seems the issue was with a very old code around r2175 and you can't even reproduce this issue on 0.9.8. and I rather doubt it's related to the following code as issue stated above seems to happen only with upscaling, so It's pretty misleading in my opinion. well, if you still want to have this at code I'll put it back.


Bravo !


Cool, I will definitely remove the email address.


Great, I think there no more left to say about it. Thank you


@martinklepsch feels much better now :)
thanks for the quick change


This has quite inconsistent behaviour on various PHP versions: https://3v4l.org/jLsM8


Looks mostly like dead code, I'd say enable it and let's do some cleanup.

`no-else-return` makes sense and only has 4 occurrences.


Merged Thanks!

Released version 0.1.4
https://rubygems.org/gems/babymetal


This removes the "Ultimaker 2 Extended +" option from the Add Printer dialog


This removes the "Ultimaker 2 +" option from the Add Printer dialog


Why not just call it `IOBuffer`?  That is what its constructor is called.  Just have
```jl
abstract AbstractIOBuffer{T<:AbstractVector{UInt8}} <: IO
type IOBuffer{T} <: AbstractIOBuffer{T}
   ....
end
```
?


@madskristensen What was the reason for this?
VS project names are not by default in lower case only. This rule is now causing warnings in all projects that have a package.json such as cordova, web, etc.


this commit doesn't seem to change anything :/


though the commit message is good!


I just remembered that we pass on our own defines via kodi.config:
https://github.com/xbmc/xbmc/blob/master/project/cmake/kodi-config.cmake.in#L19
linux example: ADD_DEFINITIONS(-DTARGET_POSIX -DTARGET_LINUX -D_LINUX -DBUILD_KODI_ADDON)


Woah, I was really tired when doing this. Thanks @Gallaecio :)


d


Oui! Cf d'ailleurs sur République numérique les propositions de [Cellular1988](http://www.republique-numerique.fr/projects/projet-de-loi-numerique/consultation/consultation/opinions/section-1-protection-des-donnees-a-caractere-personnel/action-collective) et du [CNNum](http://www.republique-numerique.fr/projects/projet-de-loi-numerique/consultation/consultation/opinions/section-1-protection-des-donnees-a-caractere-personnel/ouvrir-une-action-collective-sur-la-protection-des-donnees-personnelles), ainsi d'ailleurs que la mienne en complément du CNNum et qui vaut ici aussi: [s'assurer que tous les salariés soient représentés ](http://www.republique-numerique.fr/projects/projet-de-loi-numerique/consultation/consultation/opinions/section-1-protection-des-donnees-a-caractere-personnel/ouvrir-une-action-collective-sur-la-protection-des-donnees-personnelles/versions/s-assurer-que-tous-les-salaries-soient-representes) par la formulation:

"3° Les organisations syndicales de salariés, lorsque le traitement affecte des salariés, y compris si le syndicat n'est pas présent dans l'entreprise concernée"


nit: Spelling


We possibly create a lot of copies of the `chihaya.AnnounceRequest` by passing it as a value. Maybe we should define a few methods on `*AnnounceRequest` so we can pass it as a pointer?


@Cracks777 You will need to re-download libraries: https://github.com/TCPR/Fixes/tree/master/Thermos-Libraries


Thank you!


Why not use the getNames() function provided by LWJGL? See http://javadoc.lwjgl.org/org/lwjgl/glfw/GLFWDropCallback.html#getNames(int,%20long)


None I can think of. But you can achieve quite similar result with this, I guess. Not as "concise" as JS's `reduce` in your example, though.

http://php.net/manual/en/function.array-map.php


Excellent idea! I didn't realize this was possible. :-)
It seems easily accomplished, too.

I also noticed forks by ale and MSC who have made interesting changes, some of which I will consider importing (with some rewriting).


I propose calling this "filesDropped()" to be more in line with the other callback names.


How do we need this? Same for `handleScrape`


Thanks for your pull request. Thought the owners in the hotspot_cache are already unique.
Unfortunately there is an obvious sql syntax error: in FlickrDatabaseDatasource line 83.


Em projetos grandes, não é legal deixar arquivos grandes diretamente no servidor. Geralmente, espaço em disco em servidores de aplicação custa caro, porque esses discos precisam ser rápidos (por causa da aplicação).

Em geral, esses arquivos ficam em algum serviço de armazenamento na nuvem, como o S3 ou o Dropbox. Vou pesquisar algum gratuito para o projeto de vocês.


Let's say we have a product `x`, so we want to tell important stuff on the homepage.
- how to download
- what is it for
- who build it
- blabla

But that needs to be short (mobile first etc).
see for example this "design" http://materializecss.com/templates/starter-template/preview.html
The long explanation is in the `pages` (every block would have a link like readmore>)

There is no way to generate the short-content from the pages so you need a custom (hand-crafted) index.html

But perhaps you want to use the other butterfly-stuff like  `<butterfly-tags />` or perhaps the first 10 `posts`.
I don't need that, but perhaps a reason to reconsider.


Ah, their the missing space is :space_invader:


1.0f and 1.f are just two ways of writing the same number.


检查了一下格式和链接，没有发现什么问题。不过内容格调很高啊，要好好拜读一下。新年快乐：）


i like it!!


Nice work, Toya! Your comments are clear and convey exactly what the code is doing. The only thing I have to add (and I've made it for everyone who has completed this assignment so far) is that it helps to tell a story when you're commenting. I think of it almost like writing pseudocode so that anyone, even someone not familiar with apex syntax, can look at the code and know what's going on.


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


[array_column](http://php.net/array_column) appeared since php 5.5.0

we have misc::collect doing something similar


altho there exists polyfill: https://github.com/ramsey/array_column


Just so, within [pg-promise](https://github.com/vitaly-t/pg-promise), SQL injection is counteracted through the use of [SQL Names](https://github.com/vitaly-t/pg-promise#sql-names), which is far more efficient.


谦虚使人进步，骄傲使人落后。只有实事求是，才能与时俱进，才能谦虚谨慎。


Thanks Otto! This is great!


AddPerformanceTestForOctree (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1536 of 1536 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.71%25-green.svg)


Great work, Toya! I'm making a few inline comments above so it's easier to see what I'm talking about


its not /updatescripts, its /updatecommands


Why underscore?


> because using --- is perfectly valid markdown syntax.

Fair enough, but you're maybe the second person that made a comment about it since I created the package.


@fromgate please update plugin in forums


This comment applies to lines 58-60 and this line. What you've done is technically correct and something most of your co-learners have done as well. While this works and you will see a value in the debug logs, it will be out of context because you don't know what that statement is telling you. It's very helpful to print out a statement with the value you're debugging so you or anyone else debugging your code knows what's happening. For instance, system.debug('tell me the integer I assigned j in line 63: ' + j).


As I discussed in class, the mechanics of advancing to the next question ought to be factored out rather than replicated.


Possibly it should be named **stop-on-error**.


that said, we should probably just remove highlight support for `---` as a horizontal rule...


Nice work! You're one of the few learners who used the correct variable for this exercise. It would have been great to see you assign your integers to a variable instead of hard code them to get some additional practice with primitives.


CF11 dont like named params for DirectoryList()..
will fix it
directoryList(listDirectory,recurse,listInfo,filter)
To
directoryList(expandPath('/cbmongodb/tests/assets','false','name|all', '*.jpef')


TeamCity LabKey :: LabkeyModules15.2 :: CDS postgres 9.4 [Build 50](https://teamcity.labkey.org/viewLog.html?buildId=303173&buildTypeId=LabkeyModules152_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 22 Build time: 00:36:57
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBinnedPlot: <no details avaliable>

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

```


Stupid question: Envenom will put a DISPEL_POISON buff on the rogue, it used to be dispellable by friendly players, which was fixed. Doesn't this affect it again?


Right you only need to store the string that is the path to the file not actually open the file yet. That will come in a later assignment


This fixes exactly that issue (#15321).


Yay, more F#s.


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


Da, razmisljala sam o tome. Ispravljeno.


Oh im so stupid, sorry :+1:


The `x-forwarded-port` should be used with a property `x.portTrustProxy` rather than `x.clientPortTrustProxy`


The more traditional way is actually to make this initial judgement call based upon the value of `argc` which keeps a count of the number of arguments. If you refer to the assignment instructions the first or index 0 element is always the path the file executed at so its size will always be at least 1 for every execution.

so if we have properly passed an initial argument then the size of the arguments array as tracked by argc will be 2. Of course we do not know if we have the right argument passed but at least we can check it in this case without overflowing the argument value array bounds


Thanks - not sure why did I have to ask about it; perhaps I was just curious :-)


Nice work, Tami! Your comments convey exactly what the code is doing. A comment I've added on many assignments is that it helps to tell a story with your comments. I try to think of it almost like writing pseudocode so that even someone not familiar with apex syntax can read them and understand what's happening.


Notice how similar prevQuestion and nextQuestion are. This doesn't present as clear-cut an opportunity for refactoring as when you have two copies of the exact same code, but still, you might consider whether to generalize from these two to a single method that can move in either direction depending whether it is passed -1 or 1.


Toto appID je vybavené pre tento plugin, nieje to FYKOSí appID.
Ano, ide to dať do konfigurácie :)


Good idea with the parameter.  A couple unit tests were failing.  Run gradlew -Dtest.server=sqlite -Dtest.client=sqlite -Dtest.root=sqlite test.

You can debug tests in Eclipse if you set the databases in db-test.properties


This key seems to fail. Is the Feed table missing? Cannot seem to get it to run.


@TikhomirovSergey

Please look into this. https://github.com/appium/appium-ios-driver/pull/96#event-549189410

@imurchie has already merged a valid solution for this.


Hmmm, I messed up my commits. Didn't even notice :frowning:.


Good point, we should inject the Tor instance.


You're boolean statement here is correct, however, the request was to assign the result to the result variable, not a new one. I know this seems nit-picky, but assigning a value to the wrong variable can cause a lot of problems and confusion later on down the road.


I love this! Nice work on the primitives exercise. We'll have to share your RAD Women say... with the rest of the group.


I'm going to try to remember to ask you about your comment on line 59 at the start of class when we review the homework. If I don't address it, please do bring this up for the rest of the group, I think it's an important statement/question to talk about.


I've made some tests. Here is results from django debug toolbar (total time).

![screenshot from 2016-02-13 17-30-49](https://cloud.githubusercontent.com/assets/2611457/13028159/a8c2fec8-d277-11e5-8862-a121c722f37c.png)

AFAICS, there is no any major overhead. Feel free to post your opinion.
I also attached testing template.
[index.txt](https://github.com/dborisov/django-sitemetrics/files/129187/index.txt)


Ah, but a data container works too! That's why the message is intentionally vague. `JsonIteratorNextValueByType()` and the other JSON iterators work with arrays or maps equally well.


never commit form a text editor....


I added forum (line 12), international mail (lines 37-39) and headquarters (lines 45-50).


@turtleli
Rama triggers a crash when you hit f1 (savestate)


gf


g


Generally the more "if" statements a program has, the harder it is to understand and to test because you have to consider the alternative paths. Therefore, I generally try to see if I can find a more general computation that will work in both cases, so that I don't need to make the distinction -- I save "if" for those cases where I really need to do do quite different actions. In this particular case, you can avoid the "if" by always taking the current index minus 1, but then adding the length before doing the remainder operation so that you are sure to get a positive result. Note also that this would make the previousQuestion method quite similar to nextQuestion, at which point you might consider generalizing from the two of them to a single method for moving the current position that can move in either direction depending on whether it is passed -1 or 1.


解决与gameserver连接时，接收512个字节的问题


Why? We are not going to use any custom variables in the template.


On this line you are assigning the same value to mCurrentIndex twice, which just makes the statement harder to read without changing its meaning.


I am totally agnostic as to what is the correct coding style, but I think having one is to our benefit.


I agree, I'm not a fan of hello world either. I know you printed out your individual strings under problem 3, but here we were looking to see you do something like system.debug('This is how I feel about this exercise: ' + myStringList).


Is there an issue tracking system where these issues can be tracked?  Is there a place to list detailed requirements for these changes. In particular, in order to enforce contributor agreements, we need to user to see the a box at the top of a PR that lists the conditions for contributing, and then force the user to click "I Agree" or "I don't agree" and the the PR can only be created if the "I Agree" button is clicked.  That is needed from the legal perspective from what I have seen to ensure that the user has consented.


When printing to the debug log, it's extremely helpful to include a description of what it is you're printing. That way you, or anyone else who might be doing the debugging/troubleshooting will know why that value is appearing in the log.


Nice work, Tara! I added my comments inline in the hope that they would make more sense that way.


It broke the build... https://delivery.stups.zalan.do/go/pipelines/value_stream_map/planb-revocation-build/16


I understand and appreciate your concerns around maintaining API compatibility and to keep support for clients doing a if `error === undefined` check, but I'm not super sure about the `this._authCallback( true, undefined, undefined, message.data[ 0 ] );` approach. There are a few things that it can't prevent, e.g.

```javascript
(function(){
	arguments.length === 2 //true
})( undefined, undefined );
```

and as the first argument of the login callback already is `<Bool> success`, users checking if the second argument is `undefined` to determine if its an error seems a bit odd. I'd much prefer to just make the data the second argument.

As the data that's returned can be of arbitrary type (String, Bool, Number etc) it would also be good to use deepstream's type serialization mechanism.

So the server would need to run the login data through messageBuilder.typed()
https://github.com/deepstreamIO/deepstream.io/blob/master/src/message/message-builder.js#L63

and the client would need to restore it to its original type, using messageParser.convertTyped()
https://github.com/Svenskunganka/deepstream.io-client-js/blob/master/src/message/message-parser.js#L55


get_current will use SITE_ID if it is defined, but many django features will be broken if it is not.
Because of that we need to use default site id, e.g. SITE_ID = 1 and detect current site by our selves.


![2016_02_04_01_37_40](https://cloud.githubusercontent.com/assets/11952646/13028236/d7cf5b2a-d29b-11e5-8973-03c4a4c3d274.jpg)
# Đây là một thẻ <h1>
## Đây là một <h2> tag
###### Đây là một <h6> thẻ


No caso eu estou me referindo ao plano


Une avancée vers la transparence de la CNIL, comme indiqué lors des [débats sur République numérique](https://www.republique-numerique.fr/consultations/projet-de-loi-numerique/consultation/consultation/opinions/section-1-protection-des-donnees-a-caractere-personnel/article-17-missions-de-la-cnil/versions/rendre-public-l-avis-de-la-cnil-sur-les-projets-de-loi).

Nous n'avons toutefois pas retrouvé l'amendement imposant la publicité des avis sur les projets de loi. Il aurait fallu aussi l'étendre aux décrets & arrêtés.

Enfin, pourquoi se contenter d'une saisine facultative? La CNIL doit travailler... Plus généralement, on regrette que cette loi n'ait pas été saisie comme opportunité de revoir de fond en comble les missions et pouvoirs de la CNIL, ainsi que sa composition désuète (cf tous les [débats sur République numérique](http://www.republique-numerique.fr/projects/projet-de-loi-numerique/consultation/consultation/opinions/section-1-protection-des-donnees-a-caractere-personnel/article-17-missions-de-la-cnil), dont, par ex:

- : [publicité des décisions individuelles](http://www.republique-numerique.fr/projects/projet-de-loi-numerique/consultation/consultation/opinions/section-1-protection-des-donnees-a-caractere-personnel/article-17-missions-de-la-cnil/versions/assurer-la-publicite-de-ses-decisions) de la CNIL ;
-: [supprimer le "sauf si ce dernier s'y oppose"](http://www.republique-numerique.fr/projects/projet-de-loi-numerique/consultation/consultation/opinions/section-1-protection-des-donnees-a-caractere-personnel/article-17-missions-de-la-cnil/versions/ne-pas-contraindre-la-possibilite-de-demande-d-avis-par-le-president-d-une-assemblee-parlementaire), malheureusement conservé dans la [mouture actuelle](sauf si ce dernier s’y oppose.
- une [publicité intégrale & véritable sur tout avis de la CNIL sur un texte législatif ou réglementaire ](http://www.republique-numerique.fr/projects/projet-de-loi-numerique/consultation/consultation/opinions/section-1-protection-des-donnees-a-caractere-personnel/article-17-missions-de-la-cnil/versions/rendre-public-l-avis-de-la-cnil-sur-les-projets-de-loi) ;
- [prévoir la saisine de la CNDP et intégrer la présence d'associations dans le débat éthique ](http://www.republique-numerique.fr/projects/projet-de-loi-numerique/consultation/consultation/opinions/section-1-protection-des-donnees-a-caractere-personnel/article-17-missions-de-la-cnil/versions/precisions-sur-l-ethique-et-le-debat-public) (ce qui était[ l'objet de l'amendement n°284, malheureusement mal rédigé et donc hâtivement retiré](https://www.nosdeputes.fr/14/seance/6102#inter_13d8b1009a7ee35d971385d77ac5f8ef) ;
- ouvrir la [possibilité d'une saisine citoyenne de la CNIL](http://www.republique-numerique.fr/projects/projet-de-loi-numerique/consultation/consultation/opinions/section-1-protection-des-donnees-a-caractere-personnel/article-17-missions-de-la-cnil/versions/saisie-de-la-cnil-par-une-initiative-citoyenne-ou-associative) ;

- revoir largement la composition de la CNIL (par exemple à l'image du Haut Conseil des Biotechnologies), afin d'intégrer les associations et de l'ouvrir à la société civile, plutôt que d'en faire un repaire largement technocrate et ou d'experts fonctionnaires (conseillers d'Etat, etc., dont la culture de l'Etat explique très certainement la bienveillance de la CNIL envers la société de surveillance construite ces 20 dernières années).


ne manquerait il pas une balise span fermée sur cette ligne?


Well we invented the TARGET_FOO defines ourselves but looks like kodi-config.cmake takes care of that. Thanks.