ok, so good to add a note to this effect.
Alerts handled separately, this is just the ACK
I think ideally that wouldn't be separate...if ACK is enabled then the message containing the Alert should also contain the ACK request.
I'm in two minds about whether to send the checklist separately (so that it doesn't delay Alert & the Alert only has a single Action required) or whether to send together to have all in 1 message.


In case of a bug (like on windows) we don't want this to trigger an infinite loop. Each time we recurse we should increment a counter or count down to zero. Once we've exceeded the maximum depth amount of searching (100?) it should abandon trying to search for the build tool files.


start => inicia, emto dos os jogos né?


Ideally that's what I wanted to do, but the side panel would have to go within the chart div, and when the graph loads it currently replaces everything, so it would get rid of the side panel. For the sake of getting this project finished quickly I don't think it's something to worry too much about unless @rogeringle wants us to? I know he's time and budget conscious on getting it finished soon


Jogo ao inves de Main, melhor né?


Decrypt isn't always called, at least not from my reading of the code.  What about the call to MakeSignature and the usage of buffer, offset, and count directly in this method?  In general I'm more concerned with such asserts in cases where we're using either ```unsafe``` code or P/Invokes (or both), but such validation in other cases is still valuable, and as asserts doesn't cost anything extra at runtime.  It also helps for a reader of the method to understand the preconditions the developer of the method was expecting.


constante, estatica mesmo, vamos ensinar no final


Missing space before `` `$watch` ``.


Yes, but lookup all in bulk before the loop...just like for user_id<>pe_id etc
A single .belongs() query is vastly more efficient than 20x ==


String linha
toma mais cuidado com os nomes das variaveis, ja esta falando que eh do mapa


vai ficar new Heroi(linhaDoHeroi, colunaDoHeroi)


nome do metodo ficou maiusculo


valor


В докстринге для такой функции неплохо описать алгоритм ее работы.


@Kozoriz , Why? If someone will do some task in Genivi, he will be know about this task.


estaDentroDoMapa(l,c) faz mais sentido ne? tenta sempre se perguntar se o nome do metodo faz sentido de acordo com o q ele devolve


0, 1, 2, 3 not 0, 1


> That would actually cause an exception in the ctor.

Yes, but it would also fail if the instance were double-Disposed, e.g. if a non-null name were supplied to the ctor, then the first call to Dispose would set it to null and the second would then hit a NullReferenceException trying to dispose it.


ele nao usa a variavel heroi, entao nao precisa receber. quem esta passando veja se tambem precisa da variavel, se nao, remove


droga :(


cria variaveis int[] ESQUERDA, int[] DIREITA etc. elas sao usadas em dois lugares desse codigo. ai ja vai ajuadr horrores


:smile_cat: :+1:


desloca?


Ok, so we need to send an email to golang RHEL maintainer to ask godep packaging
And/or add a _Godeps/bootstrap directory with godep inside ?


eh uma explosao? o nome do metodo nao ta deixando claro


Fix #5 User Can View Sponsors. Placed sponsor image at the top of the footer, which stays static while views change directly above this element in the center of the app.


private se for o caso


it's Hava error


kompiliert nicht mit clang

folgende Konfiguration muss in [package_env.conf](https://github.com/MOSAIKSoftware/gentoo-server-config/blob/master/package_env.conf) eingetragen werden
```
sys-boot/syslinux gcc
```


I am a nice comment


更换主题，强势刷屏 (。・`ω´・)


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


thumbs up!


We can leave this one for follow-up after the PR.  But we should definitely revisit this, here and in SslStream.  Ideally the raw error would be passed out as part of what's returned, so that it can be roundtripped with perfect fidelity.  If that's not feasible for some reason, we at least need to do due diligence in translating all likely errors into their corresponding SecurityStatusPal, and not just blanketly translate them all into InternalError.  It's made diagnosing failures on SslStream quite difficult, because everything just shows up as error 12, the value of InternalError which is meaningless to OpenSSL when 12 is passed to ask it for a description of the failure.


Merci =)


If `call` supports that, it's the best solution, yes. :smiley:


@lgalfaso regarding "Given that we copy $$watchDelegate then there is a chance that $$watchDelegate is inputsWatchDelegate and then if inputs is undefined we will throw here https://github.com/angular/angular.js/blob/acfda1022d23ecaea34bbc8931588a0715b3ab03/src/ng/parse.js#L1865"
Its not only a potenial bug, we have a product based on angular failing all over the place because of this (worked well with angular 1.5RC).
+1 for a fix.


구글 C++ 관습을 지키려고 노력중...


Thank you, that makes sense. I wasn’t thinking of the “optional hardware” case.


Checklists definitely don't check intersection: that is subscription.location_id == template.location_id
For ACKs, this depends on whether this is the same message as the Alert or not.
Intersects are costly to compute so you should definitely only do that once.
Doing it once for the Alert & then again for the ACKs would be terribly wasteful.
As I said before, I also believe that the people who are ACKing would have this in the *same* message as the Alert.
The way you have it, the ACK request just says 'ACK the Alert'...err, ehich Alert? What if there are multiple? What if the Alert message wasn't received? What if messages get received out of order?
I think the best UX is that they receive an Alert with the request to ACK embedded within it - as part of the actions required. I'm still in 2 minds about whether the Checklist is also embedded. For an email I think it actually does make sense to embed it, but this ACK request seems to be purely SMS currently (which runs counter to our idea that users should choose how they receive Subscriptions: Email/SMS/etc...if they ask for email, all comms should be email, if they want both then they receive both, etc)
NB I would ideally put the checklist in text form into the message too...sure they have the link to get onto the website & mark items as completed...and we want them to do that, but if they cannot for some reason, then at least they still have the checklist to hand & can action items (& potentially radio status through)
Of course if this is an SMS instead of an email, then we are much more limited...but I would be focussing on email with SMS as a backup option.


Note: Travel time will be calaculated based on booking time, rather than the other way around


remote_address => from_address


Closes #1


what's exactly the point of this?


or this?


I've no view about explanations. Let's tackle Gregor.


we don't have a join point returning Boolean for rpc request task before because we already have such one for on_rpc_call, which have the same effect with this one.


wtf is that?


acordo*


If the server crashes, it recovers itself


## This is a marked file


There is an overhead for assigning io_session (ref_ptr). In this case, since we don't forward reply or reply a reply anymore, I guess we can avoid this overhead.


this isn't specific to a _docker_ bridge, is it?


@ANiknejad, has this been corrected?


Crowdin sync?


the problem was caused by API query itself, but thanks anyway.


Here is the issue, we get SO many support inquiries because users are not able to find this information in one place. I think there is no harm stating what the two options are, then at least they know where to look if they prefer to use a tooltip instead.  Are you okay if we leave this?


Just testing


Yes, that works too.


I think our approach to anything that potentially changes behavior should be to put it behind a flag, release it and watch for a couple of releases and see what/who it affects. If the fallback is just for the reason @jyotisingh said, I'm happy doing that for this. But, I think we should do some code spelunking and find the issue that this was added for. Unless you've already done it.

Then, we should not forget to remove the flag and the code.


Everything in ES6 has been "standardized" insofar that it's now a CR (big Annexes and all to describe diverging behavior of web browsers).
A lot of ES6 stuff seems to be just sugar to allow JS programmers to use more shorthands than anything else, all of which can be done in JS itself already (jsm promises for example are self-hosted in JS).
To be honest, I'm having some trouble wrapping my brain around a number of these wooly abstract concepts, and don't really understand why they would be needed to begin with for a scripting language. OOP is all nice and good, but JS is already event-driven, so why go out of your way to make abstract constructs for async operations and the like when you're not basing on a sequential language...?

So.. generators will be looked at, but I can't promise anything at this point. I want to get a solid base in Goanna first.
If you think you can help on the ES6 front I'm of course very interested.


Impatient? ;-)
Done!


مرسی داری داش


This won't work - as soon as you have a non-user principal it'll match for any person that's not linked to an indico user (`None` == `None`). You need to check the `type` too and only check email for email principals and userid for user principals.


only http://yify.is/index.php/ should be replaced with http://yts.ph


Atualizar a palavra *coperação* p/ **cooperação**.


Запятая не нужна


Check if you get that warning in dmesg at boot.
https://github.com/dorimanx/DORIMANX_LG_STOCK_LP_KERNEL/commit/384460e5647b3a9e3e314861ae93aaa5eb8d24dd


Does this work for anybody? Can this be reverted or fixed?
Breaks for me too.
See also #108, #105.


И тут title case


this is good!


Only checking access is not enough. You should actually verify that `errno == ENOENT` when you want to accept only a non-existent configuration file


Can you show me where?  I'm seeing that the SafeDeleteSslContext ctor has an assert which validates that the credential passed in is not null:
https://github.com/shrutigarg/corefx/blob/540fd0b995b1248c14742f13e9d63d96decfb585/src/Common/src/Interop/Unix/libssl/SecuritySafeHandles.cs#L272
It then tries to dereference the credential and will throw a NullReferenceException:
https://github.com/shrutigarg/corefx/blob/540fd0b995b1248c14742f13e9d63d96decfb585/src/Common/src/Interop/Unix/libssl/SecuritySafeHandles.cs#L277


As you were proposing IRL, it would be great if the message contained a link to open the same query into INSPIRE.


Это исправлено следующим реквестом.


the `which ..` part is not necessary with the ` ,, || echo ..` in place.


The comment description doesn't match the code.


這兩個方法都會重做新的instance，而不是用原本的ViewController。


Так это заголовок экрана.


Nice. I was looking for something like this last night. Should some of these points be added to the [contributing documentation](http://docs.wagtail.io/en/latest/contributing/index.html) as well? Perhaps if this was a reST document instead of a Markdown document there might be a way to include it the sphinx build so there is no duplication.


add logo gaea


hm, this seems like a caching issue. you should _really_ `rm -rf $YOUR_OPM_BUILD_DIR` and rebuild all modules from scratch...


This commit deletes all changes which were made by dorimanx for this governor.
I don't recommend it for merge, as changes which dorimanx more important than upstream changes.


Я к тому, что неконсистентно с другими языками.


Aren't the old and new code perfectly equivalent?
to me they are ...

Cosmin

On 02/03/2016 10:38 AM, Troels Henriksen wrote:
> Fix use of head that broke fusion.
>
> @coancea, do you remember why the code was written this way?  Am I
> breaking some assumption with my change?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/HIPERFIT/futhark/commit/c5d86ab8e0c144dffa4e7e144c30eb8e491f5239


typo : starTree -> startTree


В каждом языке свои традиции. Консистентность нужна внутри одного языка, а не между разными языками.


Thanks


em nenhum momento falei sobre modificadores de acesso, acha que vale a pena abordar esse assunto?


no livro se não me engano está assim também, por isso mantive o mesmo nome.


I don't get how this happens. We already have a lock above which guarantees only the timeout or the recved msg can take callback out for execution. For the endless forwarding case, if they don't send msg to the original client, I would consider it an application-level bug.


eu acho que ficaria melhor mesmo, vou alterar


I stand corrected.
1) 'func_name' attribute is for unbound methods(standalone functions).
2) '__name__' attribute is for bound methods(class methods).

Usage examples:
1)
https://github.com/JioCloud/AuthMiddleware/blob/master/jio_auth_middleware/__init__.py#L43-L52

2)
>>> class test:
...   def __init__(a):
...    print "in"
...
>>> test.__init__.__name__
'__init__'
>>> test().__init__.__name__
in
'__init__'
>>>


Sweet


let's put an assert here - should be an application-level bug indeed.


Nah, just that you do the crowdin push just before a new build, that ends up a bit untranslated.


this is going to help with packaging!

and it works insides of zipapps!


This is pulling in the portable version of Sprache into a .Net45 project. Won't hurt, but does Sprache have a .Net45 version which might be more optimized for that platform?


verdade, vou alterar


I see hyper now less jumps to highest frequency with these changes.
Will have judge the difference
On Feb 3, 2016 6:27 PM, "Realex-fire" <notifications@github.com> wrote:

> This commit deletes all changes which were made by dorimanx for this
> governor.
> I don't recommend it for merge, as changes which dorimanx more important
> than upstream changes.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gsstudios/Dorimanx-SG2-I9100-Kernel/commit/e7dc41bb395b14d334985f4c9b83eff98dbaae8d#commitcomment-15855425>
> .
>


tem sugestões?


Good to consider this usecase as yes, it's bound to come up :)

For me, I would say that if a person is subsctribed to 3x Lx & the Polygon intersects more than 1, then they should only get a single Alert.

However, the ACK/Checklist parts are separate (I think these should be separate flags in the admin-controlled subscription)

A person may subscribe to 3 locations, but only be responsible for one (they are FYI on the 2 adjacent, but wouldn't fill out ACKs for them), however they may also be responsible for 2-3 locations (in which case they should send out 2-3 separate ACKs &/or fill-out 2-3 separate Checklists)

If the ACKs (&/or Checklists) are embedded within the Alerts then the easiest here is to send a separate Alert per location they are subscribed to...and ensure that the Alert message (Subject and also the checklist) makes it clear which location they refer to


For now there are only (almost daily!) beta build. Before a stable build I will ask translators to update in advance.


Obrigada jovem


verdade, pisei na bola desse aí


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14355](https://teamcity.mesosphere.io/viewLog.html?buildId=122544&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 219 Build time: 00:07:09
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


The 2-3 ACKs may seem cumbersome for the user but I think it is better that they explicitly ACK that they have received the Alert for each Location within their area of responsibility.


Any TODOs? For me, this is more like a normal comment.


nesse eu fiquei pensando bastante, aí pra não perde tanto tempo coloquei esse só pra ter uma referência do que ele faz, mas vou alterar sim.


`@NotNull`


It would be great if could move Pi specific code from egl folder to Pi only folder. I don't like the mess in EGL and IMO the logic is inverted. EGL is not a windowing system, it is the glue code between windowing and GLES/OpenGL. X11, OSX, iOS, Windows have better implementations. X11 has it's own code for EGL.
If there are similarities in EGL between various windowing systems, we can do some refactoring later.
This is also pre-work for splitting off the render thread.


Don't call this like that. Use `Option`, and below just use `Options` for the `QFlags` version.


Cosmin <notifications@github.com> writes:

> Aren't the old and new code perfectly equivalent?
> to me they are ...

The old code assumed that the list would have a certain number of
elements, thus preventing the application of head to an empty list.
Somewhere, this assumption has been broken.

--
\  Troels
/\ Henriksen


Does a checklist really need a name?
The Template does sure, but the specific instance? I'd have thought we could build this up using a represent from the other attributes far more usefully?


@manast: This has already been fixed in #13871. The fix is in master and will be included in the upcoming `v1.5.0` and `v1.4.10` releases.


That score calulation...


aborda o assunto de construtores?


thanks!


ok


No problem. Just found it and wanted to know if this is intentional. Unused variables often mean, something is missing in the code below ;)


I think `SmartSet` fits here better


This setup #works to make the sub-submenu appear only on hover. #12


Verdade, vou alterar


Spelled wrong D:


LinkedHashMap?


Working, seems ok to close


Also, what do you plan to do if a folder contains a lot of mp3 files, plus a few cover art pictures? At the moment `omxd` skips all pictures when recursively adding folders to the playlist.


Now this class looks like a simple map from insns to frame


@viniciusgama are there tests for this?


Thanks for fixing the scope binding error. Binding dynamic values to the "md-step" label attribute works like a charm now.


It's kind of confusing that something which is called `printContext` does not have a type of `PrintContext`.


畫面遷移時的傳值方法(利用segue的話)
在**遷移前的Controller**加入下述method
```
- (void)prepareForSegue:(UIStoryboardSegue *)segue sender:(id)sender {
    // ViewController有複數個的話，需要做segue的判定。但如果只又一個就不用if了。
    if ([segue.identifier isEqualToString:@"StoryBoard上segue的identifier"]) {
        RegViewController *regViewController = segue.destinationViewController;
        ...... // 對regViewController公開的property做設定
    }
}
````


well... I suppose this split into two parts.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1291](http://teamcity.monogame.net/viewLog.html?buildId=14319&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 591, ignored: 6 Build time: 00:28:11


Once you make the change above, you can move this method inside the `PrintContext` type, so that one types the more natural: `xxx->printContext()->isClass()`.


@hashnuke I have gone WAY far off the reservation on my fork, because I want to be able to drive multiple remote hosts simultaneously.  But I thought you might find what I've done interesting - rather than have a singleton that sends one API request at a time, there is now a GenServer per remote server.


**Here's the commit that change the backend of news to redis**


RavenJToken has a CloneToken method that does deep cloning. Without requiring us to generate a string and parse a string


For backwards compatibility to python2:
```
try:
    import urlparse
except ImportError:
    # The urlparse module is renamed to urllib.parse in Python 3.
    import urllib.parse as urlparse
```


Why the whitespace?


不是在Event.cs    checkconditions()检查建筑势力ID吗？@luiges90


What are you doing!


I think it would be nice to have additional test with several parameters in `foo`, because that's where ASTORE/ALOAD of `block` will happen more than one time


It might be better to directly declare an `hasOptions` method inside `PrintContext` and use it here. It will be a little more readable and also doesn't expose the field.


To keep with urlparse.urlparse is better for backwards compatibility to python2, see above.


Why is this named d?


change as I mentioned below.


ok


The commit on assets/javascripts/image_paste.js according bind 'submit' does not work (for my redmine configuration/installation), the image is not found in the comment or description after the replacement.
If I remove that piece of code it works.
Please revert this change or correct the url.

```
Environment:
  Redmine version                3.2.0.stable
  Ruby version                   2.1.8-p440 (2015-12-16) [i386-linux-gnu]
  Rails version                  4.2.5
  Environment                    production
  Database adapter               PostgreSQL
SCM:
  Subversion                     1.8.8
  Cvs                            1.12.13
  Git                            1.9.1
  Filesystem
Redmine plugins:
  redmine_agile                  1.3.11
  redmine_image_clipboard_paste  1.0.0
  redmine_mentions               0.0.1
  redmine_mylyn_connector        2.8.2.stable
  sidebar_hide                   0.0.7
```


I see


To keep with urlparse.urlunparse is better for backwards compatibility to python2, see above.


WillRepeatedly will work correctly even if this method has never been called. You should add Times(2) if you are still testing `double check connection` according the comment above.


Odd indentation here


添加新项目


Please mind my notes to rabbitvcs/util/helper.py


Tsk, tsk -- I removed this line in https://github.com/cloudfoundry/cf-java-client/commit/f59b1164559e9f5db419423913aa5b7d5208afcd :-)


can't we just give an empty array of achievements?


在寫畫面遷移傳值前，最好先學習Objective-C基本文法。代入新的instance和代入參照是不一樣的，這是繼承C語言pointer的概念，牽涉到記憶體定址。這方面初學者不容易懂，但是很重要的概念。


I believe you don't need the `PrintContext` word here, `{...}` should be enough


See https://github.com/scala/scala-dev/issues/82.


If `verify` is a Boolean value `verify=True`, the value will be cast to String and consider as a path


Odd indentatation here


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


Weird: Browser console Elements/HTML tab didn't show the script tags when they were at the end of the body tag. 'View source' option did show them.


I will be happy if you can help me with the filter thing later (in the doc I see that it can be achieved in python, which is a bit easier for me)... because I'm kind of slow with code :wink:


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14356](https://teamcity.mesosphere.io/viewLog.html?buildId=122548&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 219 Build time: 00:07:01
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


Can we add a reference to the spec this matches to as it will help in future? i.e. http://docs.ably.io/client-lib-development-guide/features/#RSC6b3


Actually to make this more readable I will pull the creation of the PrintContext outside of here. Do it before in a separate named variable and just pass that one here.


Why do you think this needs to be wrapped in a `try` block?


This makes the event behave much less correct. Maybe try importing idlUtils from jsdom and do `idlUtils.implForWrapper(baseEvent).data = data;`. It does access jsdom internals, but should at least behave more correctly wrt on-page scripts.


for composite expressions using AND or OR do not use unless


It would be great to simply check that the next page has the expected number of stats which is one btw.


I think this can be moved to the achievement partial.


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


Maybe make ol `opearots` const?


underscore.string introduced map and it's clashing because it's required via mixin: https://github.com/epeli/underscore.string/blob/93dffd55851fef7a86a0679f5386a5c9a6a57710/README.markdown#underscorejslo-dash-integration and not dereferenced


The partial handles one achievement, it should not need to have all these information fed to it


This won't work if the page writes the onerror event I assume?


Redundant space


Why we need this at all?


Lots of very weird indentation issues in this section...


why do you need to move the achievement variable into a scope? it's lazy, if the spec doesn't use it, it's free.


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


AddWarningWhenDElightAndShading (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (2057 of 2057 tests passed, 11 test warnings)

 * 549 tests had: AUD diffs.
 * 11 tests had: BND diffs.
 * 2 tests had: EIO diffs.
 * 2 tests had: ESO small diffs.
 * 8 tests had: MDD diffs.
 * 8 tests had: MTD diffs.
 * 8 tests had: RDD 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)


`monitoring_token` is not getting written into the `rackspace-monitoring-agent.cfg` file. Once we delete the `rackspace-monitoring-agent.cfg` file and runs the task then it was writting the token into the .cfg file


Could you move this to the `else` clause above?


Sure, but regardless of the `errno` there would be no point in continuing on to try and open+parse the file, right?

My main intention was to reduce the level of criticality in both the syslog severity, and the wording of the message. As well as make it more user friendly.


It seems to me that here are too many spaces


@davidzzzzz Hey man can you publish these changes and re-release wix media? :3 <3


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 241](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=743&buildTypeId=XbimEssentials_XbimEssentials) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:51


@staeff
This is related to this commit:
https://github.com/plone/plone.formwidget.contenttree/commit/d3b41b43a8036eb517a98a0a57d71645ed4c81c5

It triggered a this wonderful weird error:
```
...
  Module five.pt.expressions, line 161, in __call__
  Module plone.formwidget.autocomplete.widget, line 151, in js
  Module plone.formwidget.contenttree.widget, line 261, in js_extra
TypeError: __init__() got an unexpected keyword argument 'default'
```
For this line of code:
```
              button_val=translate(
                  _(u'label_contenttree_browse', default=u'browse...'),
                  context=self.request
```


the code in block is executed some time after the main function. so the simple variables are copied and must be visited readonly. So the type of dumpResult in the block is const int, not int. you can't pass a const int * to int *. the NSObjects are retained and modify them will reflect the real object , so they are writable.


~~In~~ this way -> This way / Using this approach ...
(_in_ just sounds weird, I might be wrong though)


oops mistake... that "v" is obviously a copy/paste made too fast :) thanks for notifying! PyUnicode_AsUTF8 doesn't exist in  py2, that's why this little trick is used all over freecad...


How can I check that? Could you please elaborate more?


We should drop this, including the aliases and the method


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14358](https://teamcity.mesosphere.io/viewLog.html?buildId=122564&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 219 Build time: 00:06:58
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


user vote on forum : good idea.
in the future, the config.xml file will be a default configuration for the template, to fill a table in the database. So we'll can add parameter such as "use surveyname/sitename for brand" in a template configuration page.
Future...


in iOS SDK 9.1 , the NSObject says that -init is the designated init method. So , if you want initWithApplication be your subclass's designated method, you *MUST* override init, and calls initWithApplication.
However, you didn't.
So there are two solution:  the first, override init and calls initWithApplication in the overrided method.
the second, just don't make initWithApplication be the designated initializer.


Are we using an object as the key here and below? Does this work?


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


OK, this is not needed.  just be more friendly with some strict warning levels which may report that assign in if statement.
another solution is :  if ((self=[super init]) != nil)


ic, really forgot about that tbh


@ianbenn what kind of error do you get?


Los ojos de perro siberiano
Oliver Twist
100 años de soledad
Pedro Paramo
Señorita México


Did you ever test this? We would want this for CivTemp


Oh yes, sorry. For this latest update to work you will need to checkout the Blink package with `Pkg.checkout("Blink")`

I added new functionality that allowed testing for equality and using `if` statements and we haven't tagged a new release of Blink.jl yet.


Impressed at how concise this fix is! Tested, works great.


@codiebeulaine you're learning fast ;) :cake:


I tried that for a couple of times. Even I have done in two separate new folders with that only change, I still reproduced exactly the same problem. It is just something unfortunate.  I hope it is not something related to the cmake version (`cmake version 2.8.12.2`) or something from the system. I will try on a slightly different OS sometime later.


This is so very beautiful...


for loop is slower than sapply function. but sapply doesn't show story id. can do more research on this later to improve the speed.


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


Are you taking about WinSystemEGL.cpp? That seems generic and is used by all the other embedded platforms.

Yes, EGLNativeTypeRaspberryPI.cpp is platform specific, but so are EGLNativeTypeAmlAndroid.cpp, EGLNativeTypeAmlogic.cpp, EGLNativeTypeAndroid.cpp, EGLNativeTypeIMX.cpp, EGLNativeTypeRKAndroid.cpp.


This sentence feels a bit clumsy, how about replacing it with something like:
"This replaced the previously used `ForkJoinPool#commonPool()` which is a default executor of the underlying `CompletableFuture`. It is an important change in Java EE where `ForkJoinPool#commonPool()` is not a good fit."


Thanks! :+1:


could you add a comment here to point out that it's needed for 2.8.11?


I'm confused by this change.  If you made r0 in cm then shouldn't the line below also change?  (1e-9 to 1e-7)  Or have I misunderstood what's going on here?


What is that special character? can we use an existing symbol from g:airline_symbols?


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


Oh I see why you made it cm, since when supplying it as an argument you have to use cm.  My question about the line below still stands, though.


Для реализации state machine существуют гемы aasm и state_machines_activerecord. Можно было их использовать.


What the fuck.


lgtm


ok


-> addBrowsePermission... you mean


My friend, you probably want to use the `Compiler::MODE_XML`-constant to get it working correctly :)


I'll correct the spelling mistake in the UnitMethods.h fix/comment in the next PR!

Note, re first fix/correction. That code exists in the previous cores, but did not exist in Three. This is in the SpellEntry struct, which is different on Three to that of the previous cores. No doubt Three will likely need to be brought into line with the previous cores so that SpellEntry is the same. This can be done at a later time, once the core successfully builds.


```
... =$(crowbar dns proposal show default | rubyjsonparse "puts j['deployment']['dns']['elements']['dns-server'].length"
```


yep


Repeat is implemented.


I feel like this has happened multiple times. Should we create a sanity check for this?


can use in following way:
eclipse.che.ide.api.theme.Style.theme.getMatchingSearchBlockBackgroundColor();
and remove getMatchingSearchBlockBackgroundColor() from Style


I'd rather delete it and use explicitly std::hex in code


Yes, WinSystemEGL won't survive because EGL is not a windowing system. You need something like WinSystemPi that is derived from WinSystemGLES


Mesmo com a variavel PDS_PIWIK_TOKEN vazia?


是啊


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1267.0748ee945e7340db2c94ba0eee5cdf6e31c343c6](http://build.palaso.org/viewLog.html?buildId=68861&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 857, ignored: 7 Build time: 00:07:04


This deletion doesn't seem obvious from the commit message; can you explain it in addition to the changes below?


You are right it does not work. The object gets stringified to `"[object Object]"` and that is used as an index. This is a serious bug.


Great


This will not work in Contao 3, you should be compatible with both :)


Tracked this down to toyoty99/flot.gauge#4


FT message standard has "\r\n" as line separator as per: http://git.svc.ft.com/projects/MSG/repos/ft-async-message/browse/src/main/java/com/ft/messaging/standards/message/v1/Message.java#27


I don't see these dependencies on master neither. Do you imply that we
should add them back?

Is there any problem on master requiring you to rerun the setup assistant ?
If so please open an issue to reference the problem
On Feb 3, 2016 4:28 AM, "Natalia Lyubova" <notifications@github.com> wrote:

> after re-generating the package with Moveit!_assistant, it did not have
> these dependencies by default. Do we really need them?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ros-naoqi/nao_moveit_config/commit/166c9f3b4d7de0e1ac31d94b5af4d466ae574396#commitcomment-15854830>
> .
>


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1292](http://teamcity.monogame.net/viewLog.html?buildId=14323&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Tests passed: 92; exit code 1 (new) Build time: 00:12:19


It is related with this: https://github.com/gsstudios/Dorimanx-SG2-I9100-Kernel/commit/23bb60f82129621e6363ddcfc1aa20a90b842d8f
The similar behavior could be reached changing values in boostpulse.h
https://github.com/gsstudios/Dorimanx-SG2-I9100-Kernel/blob/master-lp/include/linux/boostpulse.h


The problem was with NullPointerException if the sequence element isn't present, but it works even without it.

About IdealRfCavityDrift - this could be merged with IdealDrift.

Actually it will work as is, all Element-derived classes will work.  It's on my TODO list.  I had tested the simulation as it is and did not want to introduce any error - wanted to do it in a separate step.

One thing I noticed on SNS lattice, you should be aware for handling structure mode:

if (structureMode == 1) {
  // This code is used to "guess" how far the gaps are apart at SNS; ESS in such cases always uses multiple cavities, so +PI is always correct
  Phis += Math.PI*(int)Math.round(2*(currentGapPosition - lastGapPosition)/(lambda*probe.getBeta()));

Ivo, where is this code?

I like the new long. phase code, but I dislike the trick in

public double longitudinalPhaseAdvance(IProbe probe) {

        // We trick the algorithm into resetting the probe's phase to the phase
        //  of this gap, which is the klystron phase of this cavity
        if ( this.isFirstGap() ) {

this could be avoided by letting the function longitudinalPhaseAdvance adjust the probe itself instead of returning the value.

I know, this is kind of a kluge.  However, I didn't want the probe modifying itself.  The idea is that the probe is simply a state object.  This isn't the correct implementation - just deferred until the proper one becomes apparent.  Anything you can think of that conforms with the Element/Algorithm/Probe architecture??

-
Reply to this email directly or view it on GitHub<https://github.com/EuropeanSpallationSource/openxal/commit/f6ab9db9b3eeabded89bdcbb42728523c9f81f9a#commitcomment-15841935>.


Your right ! But in fact it's required when DbTester is used as a ClassRule (because then truncate is only done once for the whole class).
I'll use a ClassRule, that will speed up the test


This is not necessary. After the `break`, this code is directly executed.


>> But, I think we should do some code spelunking and find the issue that this was added for. Unless you've already done it.

I hadn't looked at the history earlier but did that now. That piece of code was added to ensure the latest password from the config was being passed over to the agent. The first commit itself had the entire config being looked up instead of just the materials in the pipeline in context. The commit message or the story in mingle doesn't explicitly state the reason for doing that. But that has caused the behaviour that I mentioned earlier.

> If the fallback is just for the reason @jyotisingh said, I'm happy doing that for this.

@arvindsv - I assume you are saying remove the fallback.


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `default`.* *Related to file `babel/dates.py`, line 400.*

The issue can be fixed by applying the following patch:

```diff
--- a/babel/dates.py
+++ b/babel/dates.py
@@ -397,7 +397,7 @@
                      time in UTC is used
     Added new parameter for width to resolve bug #325
     :param width: either "long" or "short" or "extra"
-
+
     :param locale: the `Locale` object, or a locale string
     """
     datetime = _ensure_datetime_tzinfo(_get_datetime(datetime))
```


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `default`.* *Related to file `babel/dates.py`, line 1339.*

The issue can be fixed by applying the following patch:

```diff
--- a/babel/dates.py
+++ b/babel/dates.py
@@ -1336,7 +1336,7 @@
         return week_number

 #Added 'X' and 'x' in PATTERN_CHARS for to include datetime format support for them.
-#Added '5' in 'Z' list to include support for format 'ZZZZZ'
+#Added '5' in 'Z' list to include support for format 'ZZZZZ'

 PATTERN_CHARS = {
     'G': [1, 2, 3, 4, 5],                                           # era
```


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `default`.* *Related to file `babel/dates.py`, line 1340.*

The issue can be fixed by applying the following patch:

```diff
--- a/babel/dates.py
+++ b/babel/dates.py
@@ -1337,7 +1337,7 @@

 #Added 'X' and 'x' in PATTERN_CHARS for to include datetime format support for them.
 #Added '5' in 'Z' list to include support for format 'ZZZZZ'
-
+
 PATTERN_CHARS = {
     'G': [1, 2, 3, 4, 5],                                           # era
     'y': None, 'Y': None, 'u': None,                                # year
```


**Line is longer than allowed. (123 > 120)**
*LineLengthBear, severity 1, section `default`.* *Related to file `babel/dates.py`, line 1353 column 121 to line 1353 column 124*.


I think it makes sense to have the options here :+1:.

It's the following part that doesn't make too much sense IMO:

```
<br /><br />- Click infowindows are enabled with `vis.addInfowindow`. <br /><br /> - Hover (tooltip) infowindows are enabled with `vis.addOverlay`.
```

To be more specific:

1. We're describing `vis.addInfowindow` on this section, so there's no need to say `Click infowindows are enabled with `vis.addInfowindow`.
2. Why do we need to mention `vis.addOverlay` here? I know `vis.addOverlay` and `vis.addInfowindow` are kind of similar but they are different...


Sembla que SC_Esdeveniments no està definit enlloc... Falta algun fitxer?


Relying on getting matched pairs of MouseEnter and MouseLeave events seems risky... At the least, the comments should indicate that this is tracked by what events the control received, and not the other GG stuff that tests what Wnd is under the cursor.


This is for the plugin type itself to be discoverable


How about initializing the registry lazily? Have a look at `Aggregate` class for example.


Okay!
Thanks!!


Let's have plain map and address concurrency later.


@JustAkan 혹시 확인 하셨나요? 이번에 펙토리 이미지가 새로 업데이트 되었는데 태그만 eas 라 바뀌어서 붙어있네요. 불머리에는 이미 적용이 되어있나 봅니다?

https://android.googlesource.com/kernel/msm/+log/android-msm-bullhead-3.10-marshmallow-mr1-eas


@mbonneau It is, but you can't create an observable with it.  ie. `Observable::repeat(42, 2);`


Fixed in master and MOODLE_28_STABLE branches.


Sorry, I don't understand why is this needed here when it's not needed anywhere else where this file is included. Whatever the problem is, it would be better to fix it in `include/wx/gtk/private/gtk2-compat.h` itself...


Why do we have two assertions that are the same?


Yes.

But if you want to improve the wording, please think about the current wording again - in some situations it could be misleading. A failing `access()` does not necessarily mean that the file does not exist.

How about this:
```c
if (access(conf_file, R_OK)) {
    if (errno == ENOENT) {
        smclog(LOG_DEBUG, 0, "Configuration file %s does not exist", conf_file)
        return;
    }
    smclog(LOG_NOTICE, errno, "Failed to load %s", conf_file)
    return;
}
```


not sure why we have this quotation here


filter() gives a boolean value, so conversion in a list is obviously wrong. That's a bug in modernize tool.


> Den 3. feb. 2016 kl. 08.42 skrev James O. Coplien <notifications@github.com>:
>
> What I want instead is a model where I create all the objects and lay them out on a table in front of me, and then connect them according to contextual knowledge — by initializing them. That requires separating initialization from instantiation.

I'd argue that since all my objects are either immutable data or a context. This is no problem for my data objects and seeing that Iknow which objects play what times when I instantiate my contexts I fail to see when this becomes a problem. Back in ALGOL days they didn't have DCI to help solve the problem, so they used a different hammer


So what is the point of `publicPath`, do you have any specific use case? Why `relativePaths` do not work for you?


Yeah I can imagine it's often like that. But you know how it goes, one forgets about it, some new code is added in between, and 6 months later noone has a clue about what this code chunk was supposed to do :D

Though it does have its merits too sometimes, e.g. for fa05005 the `if (false ...` stuff I removed helped me to understand what I had to do to implement the feature, even though I did not reuse the code itself.


Thanks for this fix. What's this file you've added - is that important?
lib/extreme_startup/views


s/it's an object/is an object/


Just feels more readable.


please try https://github.com/akva2/opm-common/commit/531449951d3baf977d5bea6e767264b006f8f08b


I noticed this the typo "Trustore" instead of "Truststore" popping up in a couple of the unit test method names.


this should be 'if config[otherkey] == nil', as false is a valid configuration value.


Is control flow via Exceptions good standard practice in Ruby?


leave a comment


ok, beware of using DbTester as a ClassRule, I had many many problems, and didn't succeed to use it (talking about dao and ws tests)


Well you cannot install it for 3 anyway...

https://github.com/terminal42/contao-avatar/blob/feature/contao4/composer.json#L15


my preference would be to omit all the text up to this point, since i think it would be better to focus in directly on the essential information. We aren't writing a manual on typography, we are assembling requirements for Arabic typography and layout on the Web, so i think we should get straight to the typographic considerations that need to be made. If we did keep this text, it's in the wrong place anyway - it would need to be in the document introduction.


You're shadowing the argument `backend` here, which was the reason why the code did not fail. Would it make sense to use different names for clarity?


Why not? :)


please remove var_dump line


To test both cases when we save to the map and don't save. But now I think it is redundant and looks strange.


При инициализации объекта стейт должен сам выставляться в значение по умолчанию. Нет необходимости его в фабрике указывать.


Not in general, but I found it the best option from the users' perspective. We can use another approach if you prefer.


+1 go ahead


missing spaces here - need to run a spell checker and resubmit the content


@Golmote  I don't see a quick fix for this. We have to remove it immediately and release a hot fix version of Prism 1.4.1. What do you think?


But isn't WinSystemEGL also used by AmlAndroid, Amlogic, Android, IMX and RKAndroid?
Are you saying:
WinSystemEGL becomes WinSystemGLES
EGLNativeTypeRaspberryPI becomes WinSystemPi
EGLNativeTypeAndroid becomes WinSystemAndroid
EGLNativeTypeIMX becomes WinSystemIMX
etc ?
Is just renaming needed, or some change to the structure of the code?


Removed '6families.org' (for the time being)


test


This feature branch is dedicated for Contao 4 so why keep compatibility for 3?


Aaaand now we're at a point where I don't see the point in having this method at all :stuck_out_tongue_closed_eyes:


How are you able to keep after flash f2fs on system? Everytime i flash it changes to ext4. Thanks :)


All of our contao4 branches are compatible with both.


Yeah, I am aware of this commit :)
https://github.com/neobuddy89/hammerhead/commit/3000e9f1a82f9efacd5be535e66d033f99e6a635

Will check for warning.


Ugh.  Thank you for catching this.  Fixed.  fe74ec0


You can't use double. You should use functions dealing with float, not double.  I.e. if after deducing types etc the generated code calls `std::fmod<double, double>()` then it's bad. It should call `std::fmod<float, float>()`. Hence my suggestion to force the param type instead of relying on it deducing the types.

http://www.st.com/web/en/resource/technical/document/datasheet/DM00071990.pdf

If it has only single precision (aka float) and you use double it will use library routines provided by the compiler to calculate it in software. These are the flags used to the compiler:

```
                        -mcpu=cortex-m4
                        -mthumb
                        -march=armv7e-m
                        -mfpu=fpv4-sp-d16
                        -mfloat-abi=hard
```

That been said, I've never seen a benchmark comparing the performance of double vs float. @tridge may know that (the only one I found actually states the performance is similar: http://docs.swift-nav.com/wiki/STM32F4xx_porting_and_speed_notes)


Thanks, fixed.


@aapav01 then this memcpy base opt disable make non sense since there is TARGET_USE_QCOM_BIONIC_OPTIMIZATION := true


I will revert those new changes with are not needed


this is not true. no need for extension to get livereload to work


No, it was to address an issue in those files. I will go over the other files as well. Thanks for the diligence on this :)


Nevermind -- The other two truststore typos that I was thinking about are in the SystemConfigurationMigrationTest, but we're going to be remove SystemConfigurationMigration soon anyway.


That is possible, but to do that we have to shift the logic to check whether the `current_course_user` is a: student with no achievements, a staff, or is just `nil` (probably instance admin).

That will result in the view partial having quite some logic on checking whether the current_course_user is a staff (in terms of styling the entire achievement row, and displaying the achievement badge), which was why I went with this instead.

I'm not sure which is better though (cause what I've suggested does couple a few things together), any thoughts? Could do the other method and let you have a look too.


Okay makes sense. Will do this.


ok, thanks if it fails, I will change it


This makes sense, indeed.


this implies that app key and api key are the same, they are different


nice company satyamseo.com provide website google top ranking


did you do a `make install` once? or maybe you're a victim of the sibling builds feature?


That is true, didn't remember that part. Moved it in as one describe block didn't use it, but your point is valid. Will do the necessary changes.


@pestOO, because we have to prevent assignment in **NonCopy** assigner


Yes, I agree with you.


The idea is taken from https://github.com/resque/resque/blob/1-x-stable/lib/resque/job.rb#L57-L59.


Nice. Feedback:

* I would add the other `event_` fields in as dimensions as we can slice by exact event types unambiguously
* In the second cut of this, we may want to explore setting `dimensions` to `[]` and just excluding specific fields using `dimensionExclusions`. In other words, we blacklist columns out rather than whitelist columns in. The advantage is that individual customers don't have to keep updating their ingestion spec as they add in new contexts and event types...


`hunter_download` is a function, hence has unique scope. There is no need to protect variables with leading underscore.


Hi. There is no way to contact you here so my Twitter account is @picseel . Please contact me. I have some questions related to your plugin.


it's fine without them (just tried to answer your question :)

it was long time ago, now everything on master seems to be fine!


can you use
```
:issue:`ISSUENUMBER`
```
to reference PRs and issues?
When there's a PR, commit ID can be removed


I think no. The only thing I `make install` is the ert.  And all the other things are staying in their build directory. All the buildings are sharing the same installed ert though.


It's up to our callback to handle it. Commit https://github.com/infirit/blueman/commit/ddc46de39728f2c52cda517ca63d457a074a1a9e should do it and I'll squash it later when I figured out why the pin dialog broke (not related to this handler discussion) :disappointed:


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14361](https://teamcity.mesosphere.io/viewLog.html?buildId=122572&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 219 Build time: 00:02:25
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


Markup doesnt work here!


i'm fairly certain my fix will fix things. it's a combination of the complexity added by supporting sibling builds in many different layouts and a bug in cmake.


I think `forward_as_tuple` should be in `<CGAL/tuple.h>`. @sloriot, as one of the authors of `<CGAL/tuple.h`, what do you think?


done


деякі горизонтальні розміри лишилися в пікселях, бо вони до вюх, які на гілці test_passing(я всі stylesheets файли із test_passing засташив, бо на ній міняв структуру stylesheets папки), а якшо я не бачу вюхи то не можу перевести пікселі у відсотки


local m


use a html anchor


Valid xhtml with closed tags is fine: Symphony CMS include it to a xml tree, so it is easier to use it if it s already formatted as html. I think...


laravel始めたんですね！

最近はよくslackに常駐してます:octocat:


> Relying on getting matched pairs of MouseEnter and MouseLeave events seems risky…

Why?


yeah but it's a valid point! I may have installed moveit in a dirty way on my machine before testing this package for the last release (apt-get install ros-indigo-moveit-*). So it's possible that these dependencies are required if you try the package with an out-of-the-box install of ROS. I'm pretty sure I just reinstalled ros when I ran into the problem so the issue may still be there. I'll perform some testing in a few hours to make sure of it.


Tabs vs spaces.


I updated the spec to exclude some dimensions instead: https://github.com/snowplow/snowplow/commit/7206394acdae704e31e1958613743f003118e712


> But isn't WinSystemEGL also used by AmlAndroid, Amlogic, Android, IMX and RKAndroid?

That's correct and I'll ask the maintainers to do the same. Wayland already disappeared :)

> WinSystemEGL becomes WinSystemGLES

No, WinSystemEGL dies. There won't be a WinSystemGLES, We have a RenderSystemGLES.

> Is just renaming needed, or some change to the structure of the code?

Some changes to the structure are required because the current one is inverted. The platform specific WinSystemXX uses EGL to create a context and bind to GLES that is implemented by RenderSystemGLES

EGL is just some glue code. Look at iOS:
class CWinSystemIOS : public CWinSystemBase, public CRenderSystemGLES

It does not use EGL but has the correct structure.


Dette avsnittet burde vært ett avsnitt + en tabell som gjør det lettere å forstå.


Pattern repeated several times `${HUNTER_SELF}/cmake/projects/${HUNTER_PACKAGE_NAME}`. I guess we can introduce new variable like `HUNTER_PACKAGE_SOURCE_DIR`. By the way it can be used in scheme.


Tab vs spaces. Rett er 4 spaces.


This is the last version of unify that has automatic inputs for for wedding cakes.


Thx :D


@CleitonRichardson De novo, estou sentindo falta da assertiva. Outra coisa: não houve nenhum commit hoje?


Can't you provide the overwrite flag as a parameter, that defaults to "don't overwrite"?


To avoid having to make these fields mutable, could we perhaps drop the fields completely and instead rely on `m_epgTag->Genre()` directly?


sure


So why do they need to be discoverable since the original type should not matter when you do a raw JSon dispatch


This won't solve the problem I was referring to. Assume that you receive the stop message when the `JobStatus` is still `RUNNING`. Then you call stop on the `ExecutionGraph` and just at that moment the job status switches to `RESTARTING`, then you won't do anything in the `stop` method of the `ExeuctionGraph`. However, the user will receive a `StoppingSuccess` message.

I think we have to allow the state transition from `RESTARTING


usually when u do that....few days later, you get the device lolololol...i can see the end of op2 xD


Done: https://github.com/flibitijibibo/TrueCraft/commit/7476b76a9badd1f92baf43d9e485f321dbf26608


No necessary to upload it.


Thanks @tombentley!


Be careful here. A `FunctionType` is not the same as a `ProcedureSignature`. Specifically, procedure signatures keep track of the `Storage` that is used by a procedure for each parameter (register, stack etc). `FunctionType` has no idea what a calling convention is.


@kreigne  If these are not bootstrap-related, they could be in our compass-mixins repo


`is` -> `are` :)


@pestOO , we use Google Code Style and almost checks  are there


I mean, it adds the heartbeat feature `Heartbeats` to the type list.


there is a comma I missed here though: usually "nevertheless" is followed by a comma.


:sunny:


I think logic here can be simplified. No need to check for components here, i.e. for practical reason I doubt there will be per component scheme. Actually I think it make sense to remove directory with components and generate them on-the-fly: https://github.com/ruslo/hunter/issues/289


OK I can see the value in that. I'll do it like this:

```C
if (access(conf_file, R_OK)) {
    if (errno == ENOENT)
        smclog(LOG_DEBUG, "Configuration file %s does not exist", conf_file)

}

if (parse_conf_file(conf_file))
    smclog(LOG_WARNING, "Failed reading %s: %m", conf_file);
```


Hah! :)


On Wed, Feb 3, 2016 at 7:42 PM, Fran Boon <notifications@github.com> wrote:

> Checklists definitely don't check intersection: that is
> subscription.location_id == template.location_id
> For ACKs, this depends on whether this is the same message as the Alert or
> not.
> Intersects are costly to compute so you should definitely only do that
> once.
> Doing it once for the Alert & then again for the ACKs would be terribly
> wasteful.
> As I said before, I also believe that the people who are ACKing would have
> this in the *same* message as the Alert.
> The way you have it, the ACK request just says 'ACK the Alert'...err,
> ehich Alert? What if there are multiple? What if the Alert message wasn't
> received? What if messages get received out of order?
> I think the best UX is that they receive an Alert with the request to ACK
> embedded within it - as part of the actions required. I'm still in 2 minds
> about whether the Checklist is also embedded. For an email I think it
> actually does make sense to embed it, but this ACK request seems to be
> purely SMS currently (which runs counter to our idea that users should
> choose how they receive Subscriptions: Email/SMS/etc...if they ask for
> email, all comms should be email, if they want both then they receive both,
> etc)
> NB I would ideally put the checklist in text form into the message too...
>
 Currently, admin based subscribers receive email, sms with this template:
https://github.com/flavour/eden/tree/master/modules/templates/SAMBRO/views/msg.
Is there a way I can embed the ack part in this template as the
notify_check_subscription works async?

> sure they have the link to get onto the website & mark items as
> completed...and we want them to do that, but if they cannot for some
> reason, then at least they still have the checklist to hand & can action
> items (& potentially radio status through)
> Of course if this is an SMS instead of an email, then we are much more
> limited...but I would be focussing on email with SMS as a backup option.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/biplovbhandari/eden/commit/44aee3909ae890dfbe3fda6b086151bf378387f0#commitcomment-15855112>
> .
>

--
Biplov
ᐧ


Nice


???  Copy and paste error ?


I don't think that a flag is needed here.
To me this looks like a bug that probably almost no one reached because
they are not using double alerts.

Strange thing that when I was using them, I did not notice that as well.
(Maybe I was using that only on the night and was too tired to see what is
really going on).

I believe that once people are ok with us not alerting on 220 they are also
ok with us not alerting on 200.

I also don't think that people will understand the different behaviors...

Thanks
Tzachi

On Wed, Feb 3, 2016 at 12:29 PM, AdrianLxM <notifications@github.com> wrote:

> Thank's Tzachi! I will test it this weekend.
>
> As the logic was different for quite some time, should we have this
> optional?
> Something like "Extend snooze of higher priority alerts to lower priority
> alerts." (There may be better/more concise version.)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/StephenBlackWasAlreadyTaken/xDrip-Experimental/commit/af84c52ebf3f413aa9ad5ae0ed9a19be216dbde2#commitcomment-15852639>
> .
>


El vaig esborrar a última hora (les proves jo les havia fet amb Esdeveniments, ja que no tenia el post type de programes configurat. Cal esborrar eixa línia.


TeamCity Selenium POM :: CI [Build 49](https://teamcity.sponte.uk/viewLog.html?buildId=1054&buildTypeId=SeleniumPom_Ci) is now running


I see. The default DisplayGroup used to have the size of the wall as read from the Configuration, but it was "accidentally" removed in the qml2 migration process: https://github.com/BlueBrain/DisplayCluster/commit/3b80e1159d937b7096b8627ffd49c48cbb6baa00#diff-662a11b6d1f53551d452232ebd48c49bL68. Maybe it should be restored, even if it was not needed so far, to replace this sendAsync().
That being said, the deeper problem is that the Background is considered separate from the DisplayGroup in many ways (stored in config, not session, and passed through the Options) but then it still needs a ContentWindow, which then requires a DisplayGroup, a controller, etc... for rendering. Ideally it should be more separated, similar to the TestPattern, but that means adapting the Content synchronization/update step which may not be so straightforward...


Not sure if this is required - this means that any instances must belong to a variant, meaning that it will need a variant_id in the path


TeamCity Selenium POM :: CI [Build 49](https://teamcity.sponte.uk/viewLog.html?buildId=1054&buildTypeId=SeleniumPom_Ci) outcome was **FAILURE**
Summary: Compilation error: Compiler (new) Build time: 00:00:08


it is for me — it was looking in the wrong place for the templates, and rather than work out why I bodged it with a symlink (which should be harmless if it's not required):

Errno::ENOENT at /
No such file or directory - /Users/robin/src/extreme_startup/lib/extreme_startup/views/leaderboard.haml
file: template.rb location: initialize line: 219

on a separate but related matter, getting it working on a current Mac was rather annoying — it could use a little brush-up… still, had fun with it


test


@AByzhynar , 5974f86 , Done. @malirod , according to your comment, added changes to master & develop rules.


I think it's easier to use `set(_hunter_schemes_search_dirs "${_hunter_schemes_search_dirs}, ${download_scheme}")`. In this case we will have list of file paths instead of directories which still make sense.


@AByzhynar , 5974f86 , Done.


@dsyer this PR has already been merged, but we need to make whatever changes are necessary so that the configuration is idiomatic. What do you suggest as an alternative?


Have you tried putting a really long message in here?  How does  the UI scale?


@malirod , what link do you mean? checked all links.


How did you know this needed to be a span and not an li element?


TeamCity Selenium POM :: CI [Build 50](https://teamcity.sponte.uk/viewLog.html?buildId=1055&buildTypeId=SeleniumPom_Ci) is now running


TeamCity Selenium POM :: CI [Build 50](https://teamcity.sponte.uk/viewLog.html?buildId=1055&buildTypeId=SeleniumPom_Ci) outcome was **SUCCESS**
Summary: Tests passed: 36 Build time: 00:00:33


@mckll Looks good Aaron. Press ahead!


I am afraid that for some strings, you just don't what they mean unless you use and encounter them...


alright! thanks


Please remove commented code.


why?


This should not be needed


Yes, I can imagine that, but as I understand it, jack1 & jack2 are both meant to stay maintained and should rather be seen as two different implementations of the same thing instead of jack2 > jack1.

Having jack2 in the same package would mean jack2 superseedes jack1 and jack1 is meant to die. Moreover, this would mean every new install would pull in jack2, which I'm not sure is a good idea either.


Having a few Actions which are not displayed in the case of a Background Content is not a real concern for me; I'd rather have complete objects at creation time as this is less error-prone in the long run.
Not sure about the threading issue you mention, but sounds like something I have already observed (and solved with some pain). There might be a way to solve this with a moveToThread() or something, but has to be looked into.


If year change constant also will be changed or not?https://github.com/zdubzdab/schoolTest/commit/6c73dbf7e634d95ede69b2a7094f83c0bd9e9337#diff-16e243ee8d8c794c36231b869d6b9460R1


Not following...


I DO.


saw approval from hmmr
at https://github.com/basho/riak_kv/commit/83afe6d8c738b3aafc840329edd74bf348f374f0


merging basho/riak_kv/bugfix/bch/fn-type-error = 83afe6d8 into borshop-integration-1339-bugfix/bch/fn-type-error


basho/riak_kv/bugfix/bch/fn-type-error = 83afe6d8 merged ok, testing candidate = ff86e69b


글쎄요. 그 eas가아니라 얼리엑세스가아닐지요..?


I think it is about whether to include this inside the package and not about `filtering`. If we remove this entry, then the `jar` file wouldn't have the spring.factories entry.


https://github.com/Particular/ServiceControl.Plugin.Nsb6.CustomChecks/pull/8

I didn't need that and the custom checks worked


It is now redirectin to inspirehep (using inspirehep/inspire-next#802)!


When you reference the heartbeat project the feature should be autoloaded


OpenSSL applications will not access /etc/ssl by default because their openssl dir is /nix/store/<h>-openssl-<v>/etc/ssl which is read-only.

However, setting SSL_CERT_FILE takes precedence over the default cert file defined in X509_CERT_FILE by https://github.com/NixOS/nixpkgs/commit/788da6894fac5b20d183ce5afbab3bacd7ddeaca.

I am available on irc to discuss this, and willing to start a discussion on the mailing list if need be.


the name of the file should be: single-activity-metadata-schema-1.0.json


show is not a great function name.  Be more descriptive, what are you showing?


Ajout du package MySqlBackup .NET


Sim... quer dizer... não... sem Commit hoje... fazendo agora... (acabamos de chegar do almoço)
Quanto a assertiva e todo o resto, ainda estou perdido tentando me encontrar...


TeamCity Selenium POM :: CI [Build 51](https://teamcity.sponte.uk/viewLog.html?buildId=1056&buildTypeId=SeleniumPom_Ci) is now running


TeamCity Selenium POM :: CI [Build 51](https://teamcity.sponte.uk/viewLog.html?buildId=1056&buildTypeId=SeleniumPom_Ci) outcome was **SUCCESS**
Summary: Tests passed: 35 Build time: 00:00:33


Perhaps it's just historical deference, but I'm more inclined to use vaguely kludgy stuff in FO-specific UI code than int he GG base classes. To me, if the GG code offers an interface, I expect it to work and make sense with the way GG does other stuff. For FO specific modifications such as this, I'm less inclined to think that.


@Golmote I've created a PR with a fix for this #877


Up'd ubuntu image version so that Dockerfile will finish building.

Added custom server function to generate a haproxy rule of the form id_group, where the app id is /group/id. Also app_id will render app_id.


Why are you using System.err.println here?  It should be System.out.println unless you are printing error text like line 142.


Minor: I've only just started reviewing this PR, but I wonder whether doing this renaming at the end rather than at the beginning might've made it less painful, because presumably there'd have been less code to change and less regression test artefacts to change.


Follow PEP8 whitespace rules around arithmetic operator?


Didn't we agree for removing action for getting single snippet rating?
Anyway few things to consider here.
Typo in `avarage -> average` and I think it's not necessary to create variables for single usage.
If I would like to separate something here it would be `data.avg` :)

Consider using `this.getState()` method instead of direct `this.state` read I think it's more convenient for future testing and works in manner of immutability. Also wonder it this code would actually work because you're just mutating old object, not assigning new state.


Hi Author,

When can you release this new fixed to npmjs.com?

Thanks,
Man


prefer using Not implemented **yet** where relevant


I haven't read all the code, but are we guaranteed to always have a header row? If there are no headers, we don't want to insert "line_numbers" in a data row.


@mstevens83 ... The translation is done! How can we proceed the tests?


Ok, skal få force pusha det


*Knappar


@syl20bnr Could probably interest you.


I uploaded just a new version that is much shorter.

I'm not sure we need the run the godep utility in the build stage. With
this method (a small link of the sources to the Godeps/_workspace), we
still use the Godeps/_workspace that godep maintains but not the executable
items.

We can ask the golang RHEL maintainers, but this may be a long process.

On Wed, Feb 3, 2016 at 1:40 PM, Nicolas PLANEL <notifications@github.com>
wrote:

> Ok, so we need to send an email to golang RHEL maintainer to ask godep
> packaging
> And/or add a _Godeps/bootstrap directory with godep inside ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lebauce/skydive/commit/8e39c83fa0aadd5ca675a950b592d68824f9460e#commitcomment-15855060>
> .
>


...then maybe the logic should be moved to authorisations in cancancan?

define a new action, maybe `:show_badge`, and define a block that checks if a student has the achievement. course users will have manage rights, so theirs is a passthrough, as is instance admins and global admins.


Minor: I wonder whether splitting the renaming of the slug and the renaming of the flow class into separate commits might've made the history easier to follow.


unfortunately, markdown formatting is not available when inside an HTML block (element used for annotations)


Getting an error on this on Compile..

Error	1	'SteamAPI.GenericInventory.Inventory.moreStart.get' must declare a body because it is not marked abstract or extern. Automatically implemented properties must define both get and set accessors.	C:\Projects\SteamTradeOffersBot\SteamAPI\GenericInventory.cs	425	41	SteamAPI

I added the 'set;' on line 425 and it seemed to correct the issue


@curiosityflow Well done here. Press ahead!


given `''.join('mystring') == 'mystring'`, you can just drop the condition here


FixExteriorShadingSurfacesSolar (JasonGlazer) - i386-Windows-7-VisualStudio-12: Tests Failed (1739 of 2001 tests passed, 55 test warnings)

 * 536 tests had: AUD diffs.
 * 188 tests had: EIO diffs.
 * 262 tests had: Table big diffs.
 * 11 tests had: BND diffs.
 * 14 tests had: RDD diffs.
 * 4 tests had: ESO small diffs.
 * 2 tests had: MTR small diffs.
 * 8 tests had: MDD diffs.
 * 8 tests had: MTD diffs.
 * 3 tests had: Table small diffs.
 * 2 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: ERR diffs.

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


Minor: Although it's definitely good to have atomic commits (i.e. where all the tests pass after each commit), in cases like this where a lot of regression test artefacts need to change, I'd be inclined to separate the changes to them into separate commits. I think reducing the size of the diffs makes it easier to see what's going on.


Задается расширение файла (сохранение html)


@rstacruz `test.only`! Don't feel so bad now knowing that I'm not the only who did this :-)


this is not correct


@teckjon while developing the app use the non-minified version of the scripts - that will make debugging way more simple. `<script src="https://ajax.googleapis.com/ajax/libs/angularjs/1.4.7/angular.js"></script>` Note: the dropping of `.min`


saw approval from hmmr
at https://github.com/basho/riak_kv/commit/83afe6d8c738b3aafc840329edd74bf348f374f0


merging basho/riak_kv/bugfix/bch/fn-type-error = 83afe6d8 into borshop-integration-1339-bugfix/bch/fn-type-error


basho/riak_kv/bugfix/bch/fn-type-error = 83afe6d8 merged ok, testing candidate = bcf05a7c


I think you should change the string literal profile names to string constants contained in a separate class called Profiles or something like that. Then you can use those string constants both in the profile name and when it is referenced anywhere else in the code.


@teckjon Looks good Teck. Press ahead!


Should we just put a note about that in there "RxPHP repeat operator does not create a new Observable...". And then maybe an example of how to use it to accomplish the same thing?


Me fine
How are you


here as well


TeamCity Selenium POM :: CI [Build 52](https://teamcity.sponte.uk/viewLog.html?buildId=1057&buildTypeId=SeleniumPom_Ci) is now running


here


History here.. this one was partly because we used to have multiple jax-rs applications within the same war.. declaring the Application with the getClasses seemed to be the only way to correctly associate the parts of each app to the correct path space.
But when we redid the auth to use jwts a long while back, we dropped back to just the one jaxrs app, which means we should be fine to just have an empty app class and let it associate everything via annotations..


```
- local cmachines=$(get_all_suse_nodes) | head -n 3
+ local cmachines=$(get_all_suse_nodes | head -n 3)
```


I know, still messing around with StyleCI settings.


TeamCity Selenium POM :: CI [Build 52](https://teamcity.sponte.uk/viewLog.html?buildId=1057&buildTypeId=SeleniumPom_Ci) outcome was **SUCCESS**
Summary: Tests passed: 36 Build time: 00:00:32


Same history here.. app path had the extra path fragment because there were multiple ApplicationPath within the same war


I don't think this will work the way you want it to. Let's add a reference to the Code of Conduct, e.g., add

```rst
.. _code-of-conduct:
```

At the top of the file and then do

```rst
hosted at :ref:`code-of-conduct`.
```


@davidsansome @hatstand : Should this be cherry picked to the release-1.3 branch?


It looks like py2 implements maxfieldsize but not py3. Since there's the opportunity, I recommend renaming to `field_size_limit` which matches what it's called by Python's csv library.


This only modifies the nesting of the conditions.


just a minor consistency thing: I think it's weird that `async_suspend_internal` doesn't release the lock, but `self_suspend_internal` does.  I would move `UNLOCK_THREAD (thread)` into `async_suspend_internal`.


fro3clr, sps, bratishka


@americool You can imagine that doing some thinking about what data you will need and making that part of the design at the start can save a lot of work. It's worth the effort to do that sort of planning/thinking at the start of a project. When you start projects - you will do that.


Notice, he added his name at the end of the file in his branch. Therefore, since it was merged after other changes to master, git put his name on the line number from the branch. It doesn't see "it was at the end of  the file so let's put it at the end of the file in master." This is very important to remember when doing automatic merges on a file that was changed from multiple sources. Personally, I've had to reset development branches half a dozen times due to this easy to make error. If two people are working on the same file, always double check the file before committing the merge.


Nice, thanks !

However, can you remove the basepath with the version ?

I know that normally it should be done that way, but actually the code generate the url based on that, and even if we don't support previous version of 1.21, i still want users to be able to call a 1.8 or lower docker api with the generated code (there will be special case with bugs but overall it works and don't want to drop that).

Also with this, tests will fail on travis ci (as they use the 1.8 version).

Aside from that :+1:


Stray backslash


`<%=` causes the following code to output to the HTML. `<%` simply executes the code w/o output.


@americool DWMR. Good here.


yes, good point - there is only one "resource set" here


json assertions are usually safer : .body("[0][0]", equalTo("messageList"))


@omegak i think this is not how it was before you splitting the file - maybe you can check whether the new solution is likely to cause problems anywhere?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14362](https://teamcity.mesosphere.io/viewLog.html?buildId=122577&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 219 Build time: 00:07:41
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


实现了长按B键连续增加的功能。


Älä naura :wink: Omalla koneella kaljatuoppi ja tuotantoympäristöissä liekki varottamassa :smile:


Right. I changed this according to our standards. ;)


Minor: I think I would've been inclined to leave this change to later in the process once the underlying behaviour had actually changed.


The annotations here are being used both by couchdb, and by jaxrs.. couchdb is using jackson.. and I couldn't find another way to affect the ser/deser to json via jaxrs 2.0 in liberty without supplying my own provider.. which since we already had jackson in use for couchdb, seemed sensible to re-use here..

It's worth noting however that this does lead to the one entity having the same ser/deser constraints for both use cases.. which isn't a major issue here, but could be.. if we didn't want to send the same json for the pojo back to the browser that we store in the db.. (or if we wanted a json field known by one name in the browser usage via jaxrs, and another name in couchdb) .. thankfully we're happy with it matching in couchdb and in the browser via jaxrs..


Minor: Presumably this change would've required the corresponding regression test artefact to be updated. However, that change has been done in the subsequent commit. I think that's a little confusing.


This fixed a warning message that i had using the plugin facetoface on Moodle 3.


> it's a combination of the complexity added by supporting sibling builds in many different layouts and a bug in cmake.

great! (BTW: did anyone ever assert that sibling builds *saved* time?)


@ElvinEfendi why did we need this? It breaks compatibility to 1.9.3. I'm ok fine with that, we need to do it some point in future anyhow, and right now all distributions have stable version out with ruby2.0, even RHEL.

Just wanted to confirm.


I don't see the point, this block is only executed if `report_hook` is True. Or maybe you want the full `if` statement in the `else` clause of the `while` loop ?


It might have been changed during the last modifications (MM without 5-minute timers, with timers, ...) or simply be masked by one of the "smart"-features?

Someone used it as a trick a while back:
Imagine an alert at 220 and an alert at 200 that plays a "silent" melody. This means the 200-alert gets cancelled when the BG drops below 220 again and therefore will ring again when it reaches 220 again.
But I'm not sure if this really worked well.


ahhhhhhhhhhhhh see? u like themeing! bwahahaha


Yup, I'll fix that


Kevin, it looks like your Eclipse is not set up correctly to use spaces instead of tabs for indenting. Let's make sure that's fixed before you do more commits.


This is also the case in Node. The official [Buffer spec](https://nodejs.org/api/buffer.html) does not include `every`. The only array-like method it does have is `slice`.


Näin tulkitsin kyllä ;D


虽然GoAgent走了，但是我们还有XX-Net啊，感谢GoAgent这么多年来的奉献，为我们打开一片天空。


? what? :P


Hmm agreed. How can we determine if there is a header or not?


@Rox- that what you dont have camera :+1:  proprietary/bin/mm-qcamera-daemon
revert that


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/3158


Sadly these providers are required if we want to use the jackson annotations to affect liberty jaxrs 2.0 json serialization.

Without them, you'll see jaxrs serialise the field 'new' from the Player class 'isNew' method, despite that being annotated with @JsonIgnore.

Although we are adding the jackson provider jar to the classpath, it defines it's @Providers with MediaType.WILDCARD .. and the internal provider in liberty apparently defines it's providers with MediaType.APPLICATION_JSON, and the jaxrs spec requires the more specific provider to be chosen.. with the net result, you can't override the liberty built in provider with the jackson jar, because it's less specific.

Adding these 2 providers that basically delegate back to the ones in the provider jar but with a narrower media type enables them to take over from the built in provider.


Yeah sounds good, let me look into that.


Sorry for not branching, I just used the github GUI for editing the README.


bibfra.me has been updated to reflect these changes - thanks University of Montreal!


What I wanted to say is, the constant `MODE_XHTML` doesn't exist, you only got `Compiler::MODE_XHTML` (unless you defined it yourself).

I can only guess whats happening, but I think your editor won't convert `link`, `base`, `area`, `hr`, `br` etc. correctly right now (It should default to HTML-Mode, which is HTML5)

You should also be able to see an error (because of an undefined constant) if you do
```php
error_reporting(E_ALL);
ini_set('display_errors', 1);
ini_set('display_startup_errors', 1);
```

Change `MODE_XHTML` to `Compiler::MODE_XHTML` and you should be fine I guess :)


@kares I felt the same way but 'A ||= 1' has always worked so I think they were just trying to be consistent.


Minor: In order to reduce the size of the commit, I think I would've split it up as follows:

* One initial commit which basically just removed the `amount` option from Q1, `which_calculation?`
* A series of commits which removed a single unused node at a time along with its template, etc, and ideally updating at least all the non-regression tests to pass in each commit.
* Either include changes to regression test artefacts in each of the above commits, or if the diffs were still big, have a single commit at the end only updating the regression test artefacts.

Otherwise it's really hard to see what's going on.


Cool! I thought about making one for the FPC studio, but we're all caught up in that studio at the moment.


The code I gave you, I added to ESS IdealRfGap. The corresponding code in SNS model is in IdealRFGap.computeEnergyGain (without any comment):
```
// get phase at the gap center:
		if(!isFirstGap()) {
			phi0 = 2. * Math.PI * arrival_time * getFrequency() - firstGapPhaseCorr;
			double driftTime = probe.getTime() - ((getCellLength()/2.)/(bi * IElement.LightSpeed) + upstreamExitTime);
			int nLabmda = (int) Math.round(2*structureMode*driftTime*getFrequency());
			structurePhase = structurePhase + Math.PI*nLabmda;
			phi0 = phi0 + structurePhase;
			//phi0 = Math.IEEEremainder(phi0, (2. * Math.PI * (1.0 - structureMode / 2.0)));
			setPhase(phi0);
		}
```

> Anything you can think of that conforms with the Element/Algorithm/Probe architecture??

TraceWin has a parameter on RFGaps, telling if the phase is relative or absolute. Conforming the architecture RFGaps would have methods:
- `bool isPhaseAbsolue()`
- `double getPhase()`
- `double longitudinalPhaseAdvance(Probe probe)`
Inside Tracker you could write:
```
double phase;
if (isPhaseAbsolute()) phase = rfgap.getPhase();
else phase = probe.getLongitudinalPhase();

double dphase = rgap. longitudinalPhaseAdvance(probe);
probe.setLongitudinalPhase(phase+dphase);
```


TODO: remove debug code


@brentguf When you have a chance, could you review this?


TODO: This belongs to antialias


112796613_DesicDehum (Nigusse) - x86_64-MacOS-10.9-clang: OK (2051 of 2051 tests passed, 0 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)


Seems like this "bugfix" is missing tests? Would it be feasible to move the formatting into Python code? Seems like the templates are getting a bit heavy with logic.


Diff for the original solution, discussed here: https://piazza.com/class/ij20ifpjghn2tb?cid=33

`--- a/src/devices/timer.c
+++ b/src/devices/timer.c
@@ -82,10 +82,10 @@ int64_t timer_elapsed(int64_t then) {
     be turned on. */
 void timer_sleep(int64_t ticks) {
     int64_t start = timer_ticks();
     ASSERT(intr_get_level() == INTR_ON);
-    while (timer_elapsed(start) < ticks)
-        thread_yield();
+    thread_current()->wake_time = start + ticks;
+    thread_yield();
 }

--- a/src/threads/thread.c
+++ b/src/threads/thread.c
@@ -402,6 +403,7 @@ static void init_thread(struct thread *t, const char *name, int priority) {
     strlcpy(t->name, name, sizeof t->name);
     t->stack = (uint8_t *) t + PGSIZE;
     t->priority = priority;
+    t->wake_time = NOT_SLEEPING;
     t->magic = THREAD_MAGIC;
@@ -425,10 +427,18 @@ static void * alloc_frame(struct thread *t, size_t size) {
     thread can continue running, then it will be in the run queue.)  If the
     run queue is empty, return idle_thread. */
 static struct thread * next_thread_to_run(void) {
-    if (list_empty(&ready_list))
-      return idle_thread;
-    else
-      return list_entry(list_pop_front(&ready_list), struct thread, elem);
+    while (!(list_empty(&ready_list))) {
+        /* We grab a thread off the front of the ready list and check if it's sleeping.
+        If not, we return it. If it is, we put it at the back of the line. */
+        struct thread* t = list_entry(list_pop_front(&ready_list), struct thread, elem);
+        if (t->wake_time == NOT_SLEEPING || t->wake_time <= timer_ticks()) {
+            t->wake_time = NOT_SLEEPING;
+            return t;
+        } else {
+            list_push_back(&ready_list, &t->elem);
+        }
+    }
+    return idle_thread;
 }

--- a/src/threads/thread.h
+++ b/src/threads/thread.h
@@ -96,6 +99,7 @@ struct thread {
     char name[16];                      /*!< Name (for debugging purposes). */
     uint8_t *stack;                     /*!< Saved stack pointer. */
     int priority;                       /*!< Priority. */
+    int64_t wake_time;                  /*!< Time at which thread should wake up, in timer ticks since OS start. */
     struct list_elem allelem;           /*!< List element for all threads list. */
     /**@}*/`


Belongs to Antialias


The flag name would stay the same though, right? `--field_size_limit` seems odd


dsdsd


@TheGrits Yeah, atm we're fine with the SWC. But we've had problems before so I thought why not :D


You actually don't need to reassign the port numbers here. The Talons are plugged into the roboRIO's PWM ports, but the encoders are plugged into the DIO (digital input-output) ports.

Also, you should probably provide `port.a`, `port.b`, etc. Anything that [these lines](https://github.com/frc-2175/2016RobotCode/blob/master/src/org/usfirst/frc2175/config/WiringConfig.java#L50-L66) of WiringConfig will need.


@ytti, fwiw, 1.9.3 hasn't been supported in (nearly) a year: https://www.ruby-lang.org/en/news/2015/02/23/support-for-ruby-1-9-3-has-ended/

Even support for Ruby 2.0 will end soon.


Thanks a bunch, I added the myget source and will fetch it for now until the release to nuget.


	    public String[] getSkullType() {
	        return new String[]{
	                this.namedTag.getString("SkullType"),

	    };
}


Just being nitpicky, but you've got spaces in there instead of tabs.


What happens after SegWit is activated and an older node with a 2MiB MAX_PROTOCOL_MESSAGE_LENGTH receives a >2MiB block?

Also, small nit: Comment should say MiB like the one it replaced rather than MB


@xhaggi ?


Sorry it took me so long to get back to this.  Somehow I never saw the latest commit. :+1:


Wow. Nice finally I don't have to manually use brain.lua anymore Kappa


Admin:排版没弄好:joy:再来


I was suggesting renaming both to `field_size_limit` (consistency with Python's standard library seems like a good idea), but if you prefer maxfieldsize then make it `max_field_size` to be consistent with other agate arguments.


The majority of file sizes do not differ between the initial upload and following retries. I found 361 result files (out of the 10485) where the file size differs. They don't seem to differ much, usually O(10) bytes in either direction. There is also no clear picture on the time between first upload and next retry. Sometimes it's several hours, sometimes it's days in between.
Usually the Client stops to upload the file after the first retry but there is also a small number of result files that have up to 4 retries (5 uploads in total).


Just a small note: I'd not use minus-signs in constants (FreeRDP-*)


Witness data is stripped before relay to old nodes, so that can't happen.


0.commit日志每次都写笑脸真的好吗
1.你不知道可以[撤销commit](http://stackoverflow.com/questions/927358/how-do-you-undo-the-last-commit)
2.你不知道可以[使用.gitignore](http://html-js.com/article/2030)来避免跟踪某些文件
3.你不知道emacs可以通过[设置](http://stackoverflow.com/questions/151945/how-do-i-control-how-emacs-makes-backup-files)，不产生~结尾的备份文件
> ...
> Alternatively, you can turn backups off my setting __Make Backup Files__ to off.
> ...


For me, the current version is more readable. The block is executed at the end of the while loop.


It works for the first 3 restarts, then it's dead again LOL


Minor: I reckon these small commits removing a single unused calculator method at a time might've been better included in or close to a commit removing the relevant flow node. That way it would've been much easier to explain in the commit note why the method has become unused.


I suppose it will have to be passed as an additional argument. It looks like only `from_csv` accepts a `header` argument.


Oh no, I like `field_size_limit` as well. I mean on csvkit, would the flag name change? FieldSizeLimitError in agate.exceptions refers to it.


import java.io.*;
import java.net.*;
import java.xml.sax.*;
import org.xml.sax.*;
import org.xml.sax.helpers.*;

public class SAXTest
{

  public static void main(String[] args) throws Exception
  {
    String url;
     if (args.length == 0)
    {
       url = "http://www.w3c.org";
       System.out.println("Using" + url);
    }
    else url = args[0];

    DefaultHandler handler = new DefaultHandler()
    {
       public void startElement(String namespaceURI, String lname, String qname, Attributes attrs)

       {
         if (lname.equals("a") && attrs != null)
         {
             for (int i = 0; i < attrs.getLength(); i ++)
             {
                String aname = attrs.getLocalName(i);
                if(aname.equals("href")) System.out.println(attrs.getValue(i));
             }
           }
        }
     };

     SAXParserFactory factory = SAXParserFactory.newInstance();
     factory.setNamespaceAware(true);
     factory.setFeature("http://apache.org/xml/features/nonvalidating/load-external-dtd", false);

     SAXParser saxParser = factory.newSAXParser();
     InputStream in = new URL(url).openStream();
     saxParser.parse(in, handler);
   }
 }


Debug code.


Best commit ever


@mariadalfonso @12345ieee this might also explain the 19 MeV on lepton pT due to the alternative recoil model (?)


24 is enough for service's groups nums. Actually, stock just have 12 and it
works fine.


Lgtm


Yeah @varund7726 , he found his way here too lol


Oh, sorry, I misunderstood. Yes, CSVKit would keep the same flag name for backwards compatibility. However, I think we should delegate responsibility to CSVKit for adding that sentence to the error message, since it controls flag names.


Subsystem should have been `form:`.


I updated this routine to include a more descriptive scenario with sample data. This should help illustrate both the utility and methodology of the routine.


You just shared our root password with everyone.


That is a bit more complicated...
No everybody uses Pulse.
On option to tell ffmpeg which audio source to use would be better I guess.


Yes, we were discussing exactly about this earlier with @mariadalfonso .
I still wonder if this commit doesn't introduce any bug, but I've yet not checked.


I'm doing experiments with core33g's rild and libreference-ril_sp
If it works well i'll send you


I thought we would set the capacity for buildings at 15 or 16 kWp? Am I reading it wrong? It looks like it is 1.6 kWp here.


Agreed. I'll refer to the parameter in the agate exception then.


This looks messed up (see preview) due to the line breaks.


Cf. #1


This passed in travis on the branch but there is a swagger integration failure now that it tested it before deploy.

First build:
https://travis-ci.org/18F/openFEC/builds/106558584

Second, failed build:
https://travis-ci.org/18F/openFEC/builds/106598979


Ok that makes sense, so is there a service bit for SegWit?

Won't propagation of SegWit data be slow/unreliable if a large portion of the nodes aren't SegWit compatible after activation?


Ho ok, sorry I didn't understand I thought this could be done like that when I saw this line: https://github.com/Talesoft/tale-jade/blob/master/Compiler.php#L230

so do I have to change

```
$this->compiler = new Compiler([
            'pretty' => true,
            'mode' => MODE_XHTML,
            'paths' => null
]);
```

to

```
$this->compiler = new Compiler::MODE_XHTML([
            'pretty' => true,
            'paths' => null
]);
```

? Sorry to ask, I am more a designer than a dev ;-)


Yes, there is a service bit now, and yes.


@psychopy/translators this commit adds a new parameter to the Movie Component and could do with the name and label being translated


this method is verbose, the origin code  `rv, status_or_headers, headers = rv + (None,) * (3 - len(rv))`  is better, shorter and cleaner.


The target parameter should be required and the var_sources a variable argument.
See https://github.com/google/closure-compiler/blob/master/externs/es6.js#L1522-L1528.


yeah, this was never officially completed :)  So it never got a chance to see the light of day imo.


quero apaga


For me *yet* means that it will be implemented shortly.


Confirmed, these dependencies are necessary.
moveit_fake_controller_manager: needed for demo.launch (anything that doesn't talk to a real controller)
moveit_simple_controller_manager: needed for moveit_planner.launch (anything that send command to an actual controller, either dcm or gazebo)
Thanks for reminding me this discussion. Otherwise the problem would have remained


trzeba było zostawić w spokoju ten tryb


`line-height` лучше в em'ах указывать) Ответишь сам почему?


- [ ] `System.out.println(Base64.getEncoder().encodeToString(cipherBytes));` に対する説明として「暗号化された内容を確認したい場合はBase64エンコードする。」とありますが、暗号化後のバイト配列をBase64エンコーディングした値を確認することで、本質的には何が確認できるのでしょうか？ この説明からだと、何を確認したいのかが伝わってきません。


Looks good


Same comment for households. Is it 1.5 kWp or 0.15 kWp per unit. Maybe I am reading the numbers wrong.


Yyhhhh


It should no longer crash on start


As Kanye West said:

> We're living the future so
> the present is our past.

# This is an <h1> tag
## This is an <h2> tag
###### This is an <h6> tag

1. Item 1
2. Item 2
3. Item 3
   * Item 3a
   * Item 3b


shouldn't `bind_sync` be implemented by just calling `bind(...).get(ec)` to avoid code duplication?


Aha, scratch that previous remark


See https://github.com/googlei18n/libphonenumber/issues/913


You would have the decision to exit the loop at the same place where you print the last report, that's all.


It was working well in Atom.


Good job!


Why do we need to link to "http://docs.cartodb.com/cartodb-platform/cartodb-js/api-methods/#visaddoverlayoptions"?


清理掉可执行文件和中间文件：
```
$ make clean
```


Jag gillar inte 5:or.


Maybe could be useful to have this file in a proper subfolder?
Donno, something like "vms" or "setup"?


solves the #1


Use https://github.com/percona/PerconaFT  the Tokutek/ft-index repo is just a regularly updated mirror.


So users who are interested in that alternative will know where to find that information.


давай проверим идею насчёт .png→.jpg и я закрою коммит) По идее картинки для позиций каталога загружаются менеджером, а они вряд ли будут готовить полупрозрачные кругляшки... Мы джпеги можем скруглять через `border-radius: 50%`


Oh, forget it. "Emergency calls only" :D
Why ril setprop is permission denied? Need on property >> setprop?


Hey @helfrij, remember that you will need to arm the scaler first.  It'll probably need a list action as follows:

```c++
listAction->addSubAction(scanningPV, 1); // Arming the detector
listAction->addSubAction(new AMWaitAction(new AMWaitActionInfo(1.0))); // Hopefully not that long though.
listAction->addSubAction(AMActionSupport::buildControlMoveAction(softInput_, 1);
```


Weird.


各个分量（分钟、小时、日、月）的增加出现了3次，能不能消除这重复？


Ok, But how should I deal with something like `>=media-sound/jack-audio-connection-kit-0.121.3-r1[${MULTILIB_USEDEP}]`?


That `!important` looks very suspicious.


That's really strange since it's working well for me.

Does nothing show up, or is it a solid block of color?
Does the normal clock output work with 256 color mode?


9+ Hrs study today


Why "decision tree" is added @@?


but demo.lauch works without these dependencies on all PC I've tied


TeamCity Civic Assembly :: Build backend + frontend [Build 53](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=674&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


I like how the expected value didn't match up with the test name!


@modsmoss please make commits in smaller sizes. This commit affects a lot of files, and would be better as several commits.


This should be in the _behavior_ config - the behavior should not reach out to Configure.


Actually realized this line is fine - just structured a little oddly - my brain jumped right to thinking it should be ``which_try += 1``.


numbers is bad, you can use some meaningful names
tuple1 -> tuple_rv_code_header
tuple2 -> tuple_rv_header
            -> tuple_rv_code
etc.


DestroyAllPlayerProgress Bar... or Bars?


This Meteor.isClient statement is unnecessary, since the code lives within the `/main/client/..` folder. Meteor recognizes folders named client, and only sends their contents to the client.


also fixes #54


:+1:

:wink:


Change le 15 pour une constante, **```const float DISTANCE_POURSUITE = 15```**


Well yes, that can be done.


I think it makes more sense to check whether `defaultLocale` === false, rather than create a new mutually-exclusive config key


Async makes no real odds that I can see here as there is nothing interactive happening.
I don't see why you can't add an optional ACK & optional Checklist section to these templates?


This should be moved below the definition of `cartodb.CartoDBLayer.CartoDBSubLayer` (line 419/421).

I'd just say:

```
This is a subclass of [`cartodb.CartoDB.SubLayer`](#sublayer-anchor).
```

IMO, we can assume folks know what subclassing is. If you don't think so, then we can keep the link to the wikipedia article.


NB If the message sending is in a loop, then should do all the bulk lookups outside the loop & then ensure that the data structure(s) are visible to the send


Rather than setting an arbitrary timeout, perhaps the spinner can be toggled reactively. E.g. the spinner can be made to show only when data is not available, such as search results, and then disappear when the results are available. This is the great part of using a reactive framework.