Are you asking based on a thought experiment, or based on an observed problem?

I would expect that MIME-Header encoding would provide a multi-line string like:  "foo\n\x20bar".  The code above would turn it into "foo\x20\x20bar".  The next line would then re-wrap the line.

As a side note, I strongly suggest you avoid using MIME-Header encoding via Encode, as it has a number of known bugs , most significantly https://rt.cpan.org/Ticket/Display.html?id=40027

Instead, consider using Email::MIME over Email::Simple.


#ImScared


closes #144


fixed in #76


Oui, j'ajouterais un git pull dans ce genre :-)
```
git checkout production && git pull
git cherry-pick <commit>
deploy.sh
```


au cas où


You probably want to actually make a strong reference:
```swift
if let self = self where self.status != .Connected {
    ...
}
```

Otherwise it's possible that, since `self` is `weak`, it can be deallocated **after** you've already checked `self != nil`, but before this method has finished, resulting in very unpredictable behavior.


This is a wip ramdisk


ok merci !


Queries are not threadsafe, but the provider will be in most cases.
What would synchronize here improve?


TeamCity Colore :: Main Build [Build 133](http://tc.sharpblade.net/viewLog.html?buildId=676&buildTypeId=colore_mainbuild) is now running


@leovolpatto A alteração
-  public function MarcarPedidoComoProntoParaEnvio($numeroDoPedido) {
+    public function MarcarPedidoComoProntoParaEnvio($numeroDoPedido, $event_date) {

Precisa ser feita também no IPedidoDeEnvio, caso contrario teremos conflitos e não conseguiremos rodar.
 - public function MarcarDiversosPedidosParaProntoParaEnvio(array $pedidos);


Merged


Thanks, fixed.


TeamCity Colore :: Main Build [Build 133](http://tc.sharpblade.net/viewLog.html?buildId=676&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:1:42


All of that client replacement loop seems really fragile and weird to me.

But I think the point of this is to not apply the optimization if you were not able to replace all clients (in order to avoid duplicating computations, I guess).

It might be a good idea to refactor this so that it makes more sense and is not as fragile.  I'm also not convinced that it is actually useful since if replace a node it will replace its use in all clients automatically.  Perhaps this is really old code.


Why did you decide to get rid of explicit references to versions?


TeamCity ReOpenLDAP :: devel-2.4 [Build 197](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=854&buildTypeId=ReOpenLDAP_Devel24&guest=1) is now running


I'm not too learned with the math, so this is all guess work, but do you mean something like:
```
auto velo = dyn.vel;
auto angVelo = dyn.angVel;
auto normalVelo = velo;
normalVelo.normalize();

Fd = -0.5*liqRho*dragCoef*area*(velo.cross(normVelo))
```
?

My colleague only derived the equations in terms of its components and we thought |ω| was referring to the magnitude/abs value of the vector instead. They will try to rederive it in terms of vector equations.

A question, though. What's wrong (besides the abs() part) with writing it in terms of components. Is the for loop just not efficient and should be avoided or?


I know they are not threadsafe.
Maybe someone, somewhere uses executors to execute the queries, I don't know.
I just suggest it, you don't have to implement it of course.


That's something like approving a PR.  Merging to master


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


I meant moving the declaration (360/361) to just above 383, where you use them.
The other way around is fine too, keeping them close together is the important thing, imho.


TeamCity LabkeyTrunk :: Git sqlserver 2014 [Build 1](https://teamcity.labkey.org/viewLog.html?buildId=275380&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:01:22


@gdsotirov most FFmpeg devs don't read GitHub comments, so please file a ticket on https://trac.ffmpeg.org.


:+1:  :construction_worker: :construction_worker:


Doubtful, it's not the same place, and the way it works would probably
crash that anyways


closed #144


asd


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


Portuguese-Brazil language!!!

Hi Scarfmonster,

I have started the portuguese translation for yacy. I suppose there are 50% , 60 % of all translated.

How can I add my file "br.lng" on yacy/locale repository?


What about:

```
try:
    from io import SEEK_SET, SEEK_CUR, SEEK_END
except ImportError:
    SEEK_SET = 0
    SEEK_CUR = 1
    SEEK_END = 2

```


Thanks! BitHub has sent payment of  $33.47USD for this commit.


Much better, thanks.


Java actually won't allow null check on primitive values. All the other ones are non-primitive and I have to check them for nulls.


Why not just make this the default value for the `follow_path` param?


(The only thing I have left to say is that it might be worth making the connection between `def.depth` and the partially resolved path, but I think it's also fine as is.)


fixed


We should turn off input copying here.


Fixed moved it after the standard messages, now this will show Blocks in future as an additive message to standard tip.


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


@AlexanderLitus, note this TODO and fix accordingly.


The essential content of this PR is contained within PR #42. It can be postponed until decision about the other PR is made.


@AlexanderLitus, why not extract a variable assuming we're in the cycle?


TeamCity Colore :: Main Build [Build 134](http://tc.sharpblade.net/viewLog.html?buildId=677&buildTypeId=colore_mainbuild) is now running


Maybe we should match parallelism here with the input operator to enforce chaining. Also this might cause some problems if it is applied on a unioned stream


@AlexanderLitus, you set field value anyway. Notice the simplified logic. Do read “Refactoring”.


TeamCity Colore :: Main Build [Build 134](http://tc.sharpblade.net/viewLog.html?buildId=677&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:0:59


TeamCity Colore :: Main Build [Build 135](http://tc.sharpblade.net/viewLog.html?buildId=678&buildTypeId=colore_mainbuild) is now running


Could just do `s.peerConfig = &peer.Config{`.  No need for the additional local variable.  Also, could just use the locals that the server is created with instead of accessing the server fields again.  For example: `NewestBlock: db.NewestSha,`


I'll create a new issue - will move member vars to the top.


@AlexanderLitus, the converter doesn't really belong to the registry class. It is registered in an instance of this class, but it doesn't make it a good candidate for being an inner class.


TeamCity Colore :: Main Build [Build 135](http://tc.sharpblade.net/viewLog.html?buildId=678&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:0:59


good catch. I have fixed it now by adding a flag to safemode.leave and invoking that object directly.


TeamCity Colore :: Main Build [Build 136](http://tc.sharpblade.net/viewLog.html?buildId=679&buildTypeId=colore_mainbuild) is now running


fixed


You may misunderstand my meaning. I would like have a simple example app in this repo, but 2 xcodeproj for each of them. So the root dir of the project will be:
```
AUTHORS
LICENSE
NOTICE
README.md
XWebView/
XWebView.podspec
XWebView.xcodeproj/
XWebViewSample/
XWebViewSample.xcodeproj/
XWebViewTests/
```

I believe `pod try` can work with this.


Got it, thanks!


Right, but in this case we're being dispatched on the main queue, which actually should probably be the handleQueue.


initial commit


TeamCity Colore :: Main Build [Build 136](http://tc.sharpblade.net/viewLog.html?buildId=679&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:0:58


Could we import `NBFormatError` here, as well?


The loop is much less readable, that's the most important -- both for a programmer, as well as in that it conceals the physical meaning, which must be independent of component representation (invariance; an exception is e.g. numerical damping in Leapfrog integrator https://woodem.org/theory/leapfrog.html#numerical-damping, which indeed is nonphysical).
I read the formula as scalar product of one vector with several scalars (so `liqRho*dragCoeff*dyn.vel*vel.norm()`). But please get some solid reference, software manual with dubious equations (for box!) is not a good one. Just google "rotation drag spherical particle fluid" (or use google scholar).
Note that your code above (with 3 new variables `auto velo` etc creates 3 temporary copies, where you should either use `const auto& vel=dyn.vel`, or just use `dyn.vel` straight.)


\*sees Classic branch\*

TOO SOON! YOU HAVE AWAKENED ME TOO SOON...


Not sure if it would make any difference, but can't you use int64_t comparison directly here?


OK, make sense.


Could we import `NBFormatError` here, as well? Seems it should be available. ( https://ipython.org/ipython-doc/3/api/generated/IPython.nbformat.html#IPython.nbformat.NBFormatError )


My bad on (1)

For (2) -- all my commit history on our anchor fork has avoided mentioning private/proprietary projects/issues believing that good upstream hygiene was an overriding concern. Should I change this?


I am doing an commit-per-issue. Will create a sep commit/issue for this one


Can we use the release version yet?

On Thu, Oct 1, 2015, 11:02 Vladimír Čunát <notifications@github.com> wrote:

> Yes, but for now the only know work around is to keep updating the hash...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/10054a676a2e8cb21c1f5eee89a9d73166be8fb1#commitcomment-13547454>
> .
>


TeamCity LabkeyModules15.2 :: CDS postgres 9.4 [Build 25](https://teamcity.labkey.org/viewLog.html?buildId=275378&buildTypeId=LabkeyModules152_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3 (1 new), passed: 13 Build time: 00:24:44
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

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

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

```


nit: why extra newline?


LGTM


this is a pretty dramatic increase


I guess you did this with a program. Can you do the same to JSON.fr?


This does get noisy sometimes, but it can be disabled via the config if you don't like it.


I needed this in building a cluster of machines with a synchronized clock. The slaves all need to specify timings relative to the master's time, not `Date.now()`. Since it's potentially useful in a variety of situations and backwards compatible, I see no reason *not* to roll it in.


Consider: `auto&`


OK, updated.


This is from the google summer of code. This recursive replace is a speed up to the optimizer.

I think it isn't useful and so I removed it in another PR and all tests pass.

The useless replace that end up in the dict are just ignored during the replace as they replace var that aren't in the fgraph.


oh, I just saw that I guess I didn't send this as a pull request. So, yeah, if you think it would make a good addition, I can send a PR.


But that doesn't guarantee anything, the last reference to `self` can be in any thread, which means that `self` can be deallocated concurrently with the execution of this block.


Looks freaking awesome...


Please use System.Net.Http.HttpMethod instead


In the old code, it was not incremented in the "if (itr->second->bid)" case.


Haha.. Tx tx.. But too many buttons :p
On Oct 2, 2015 1:00 AM, "Praphul Sinha" <notifications@github.com> wrote:

> Looks freaking awesome...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/linux-nerd/todoapp/commit/85a3c678e2ed3fc27e85336c1c08675f9b697553#commitcomment-13549209>
> .
>


Probably correct, i'm planning on implementing entity based TNT.


first page on launch... give date as nan..nan
can u chek on that


I don't know what unit they're in, though. I want it to be around 32 nodes, it seems to either not work or use a different measurement unit.


may be bacuase i've earier taskw ith no date.. got it...


To make it consistent with create-post form (since now we have create-attachment element on the left). I can make it the same on edit-post in the timeline, to make it the same for all the posts everywhere. Will it work for you?


Why can't this simply be: `std::sort (signers.begin (), singers.end ());`?


got it.. its not a bug... pretty awesome job..thanks a ton..


I should remove them and put them into another seperate repo.


Sure. Let's also double-check this with @urbansheep.


LGTM


Fixed now.


Clear ur local storage :)
On Oct 2, 2015 1:02 AM, "Praphul Sinha" <notifications@github.com> wrote:

> may be bacuase i've earier taskw ith no date.. got it...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/linux-nerd/todoapp/commit/85a3c678e2ed3fc27e85336c1c08675f9b697553#commitcomment-13549257>
> .
>


barometer?


I need to remember why we needed those redirections


Why can't this be:

    std::sort (signers.begin(), signers.end(), [](STObject const& a, STObject const& b)
    {
        return (a.getAccountID (sfAccount) < b.getAccountID (sfAccount));
    });


@ctsims Here's the code for doing the string formatting replacement if you want to take a look. I didn't realize this PR hadn't been merged at all yet, I think it can be once this is OK'd, unless @phillipm was waiting on something else?


@satinfo  I did some tests and I must say it is very good thanks


Bhai css krne me jaan nikal gya :p
Abb bas kaam mil jaye aur baith k paisa champe :D
On Oct 2, 2015 1:03 AM, "Praphul Sinha" <notifications@github.com> wrote:

> got it.. its not a bug... pretty awesome job..thanks a ton..
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/linux-nerd/todoapp/commit/85a3c678e2ed3fc27e85336c1c08675f9b697553#commitcomment-13549266>
> .
>


OK, I will change it. I was just a little desperate at this place since we already have `attachments`, `that.attachments` and `attachmentPromises` in there. Will `attachmentList` work for you?


Isn't this line overwriting previous assignments?


Very nice.

But don't forget the `.InXScope()` methods that the Ninject documentation demonstrates. If you want singletons (like only one `IGame` throughout the life of the game), you'll want to do that.

The default (not specifying it, which is the equivalent to `.InTransientScope()`) is okay for a few things, but it means you'll get a new one every time something it is requested, and that Ninject will never clean it up (which is important to avoid for `IDisposable` things.)


Aaj ka din bda hectic tha.. Couldnt see ur algo
On Oct 2, 2015 1:05 AM, "Abhishek Prakash" <abhishekabby@gmail.com> wrote:

> Bhai css krne me jaan nikal gya :p
> Abb bas kaam mil jaye aur baith k paisa champe :D
> On Oct 2, 2015 1:03 AM, "Praphul Sinha" <notifications@github.com> wrote:
>
>> got it.. its not a bug... pretty awesome job..thanks a ton..
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/linux-nerd/todoapp/commit/85a3c678e2ed3fc27e85336c1c08675f9b697553#commitcomment-13549266>
>> .
>>
>


Sure.

Dierk

> Am 01.10.2015 um 21:27 schrieb Ingo Wechsung <notifications@github.com>:
>
> I guess you did this with a program. Can you do the same to JSON.fr?
>
> —
> Reply to this email directly or view it on GitHub.
>


+1 Really needed this option. Otherwise browsersync or watch won't reload when twig files change.


so this actually works in practice, but fails when run through RSpec because capybara misinterprets the bad certificate on thesisdeposit as an infinite redirect. I would suggest just marking this test as pending for the time being by making this the first line of the test

```ruby
pending "thesisdeposit.library.ualberta.ca getting a proper SSL certificate"


Blablabla...


I can't say I've ever seen 'mock' abbreviated this way.

At some point, you'll want to take a look at mocking frameworks - we use `Moq` in PUMP, but feel free to play with a different one, too.


I believe it was due to the fact that code was loaded/executed, thus it could write to the same streams that avocado was using.


Not from your patch, but ENOMEM is not accurate in here, sine sol_coap_get_payload() does not allocate any memory and only fails if it is provided invalid parameters.


Ah, OK, 0783738a. I need to follow up and see what happens when examples/tests/failtest_ugly.py is executed.


remove this line.


Nice!


Im a little confused still. Are you saying that an absolute path will begin with '/Users/...'?


After the previous commits some exceptions still get through.


Hey @marcoow, is 'key' configuration gone for good, or do you plan to reimplement it?


@pierremtb hmm, yeah, that's true, maybe we need some responsiveness here.


TeamCity Colore :: Main Build [Build 137](http://tc.sharpblade.net/viewLog.html?buildId=680&buildTypeId=colore_mainbuild) is now running


Don't you always set threads to 0 here?


Couldn't you read 4 bytes from `0xF3`?

```
0xF3: T msb
0xF4: T lsb
0xF5: H msb
0xF6: H lsb
```

Keep in mind that my shield hasn't arrived yet—I could be totally wrong.


Es para practicar, que ya me sentía oxidado usando git


Since your CMakeLists requires C++11 support, why was the change from NULL to 0 when the recommended replacement is NULL to nullptr?


TeamCity Colore :: Main Build [Build 137](http://tc.sharpblade.net/viewLog.html?buildId=680&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:1:0


TeamCity Colore :: Main Build [Build 138](http://tc.sharpblade.net/viewLog.html?buildId=681&buildTypeId=colore_mainbuild) is now running


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.486.3b41076b089d959d7fbd1e5e2ba7178ef88541e0](http://build.palaso.org/viewLog.html?buildId=63982&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:16:49


Simple, it doesn't run at all:

```
/scripts/avocado run examples/tests/failtest_ugly.py
Unable to discover url(s) 'examples/tests/failtest_ugly.py' with loader plugins(s) 'file', try running 'avocado list -V examples/tests/failtest_ugly.py' to see the details.

 ./scripts/avocado list -V examples/tests/failtest_ugly.py
Type       Test
NOT_A_TEST examples/tests/failtest_ugly.py
```


LGTM


So, questions - do we want minc library to try to read ~/.mincrc every time it opens a file - if I use miget_cfg_int inside miopen ?


TeamCity Colore :: Main Build [Build 138](http://tc.sharpblade.net/viewLog.html?buildId=681&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:0:58


What I think we should do is to have a class `HighestTransaction` or something like that, that has roughly the same methods as `OutOfOrderSequence` and which uses the exact same mechanism for being updated about, and getting the highest transaction, with `version`, the check loop and all that.


TeamCity Colore :: Main Build [Build 139](http://tc.sharpblade.net/viewLog.html?buildId=682&buildTypeId=colore_mainbuild) is now running


The logic behind this was - if both details are null then both get assigned new Date, so I added a second to one of them.

I am going to flush all of these changes, since all of them were made redundant by removing Null items from the collection.


TeamCity Colore :: Main Build [Build 139](http://tc.sharpblade.net/viewLog.html?buildId=682&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:0:57


Even `attaches` will do.


Throwaway example code for creating spider.  Should be removed entirely.


The new way to configure authenticators, authorizers and stores is to create custom classes that extend them and override the properties as currently already done with Ember Data adapters for example.

```js
// app/session-stores/local-storage.js
import LocalStorageStore from 'ember-simple-auth/session-stores/local-storage';

export default LocalStorageStore.extend({
  key: 'my-apps-key'
});
```


I'd say this is now already tainted, so you could leave it like this.


Jail and taxes seem like responsibilities that can be separated from the `Game` class. Consider refactoring such that `Game` doesn't stay or become a bigger responsibility magnet. (The observer pattern might help.)


Yeah would be nice to have it consistent everywhere.


notes for me [13:38:23]  <joepferguson>	TableFlipPlugin disabled for now so Phergie won't crash on characters that plugin doesn't recognize.
[13:39:42]  <elazar>	^ elstamey: Just give joepferguson a shout when you think you've got it fixed and we can re-enable it.
[13:39:58]  <elazar>	Whenever you can, no rush. :)
[13:40:38]  <elstamey>	OK!  I wish I had remembered last night when I had all that free free time :P
[13:59:29]  <elazar>	Eh, it happens.
[14:00:56]  <elstamey>	found this also: https://github.com/tightenco/symposium/blob/master/doc/api.md#conferences
[14:00:57]  <Phergie>	[http://gsc.io/u/42] symposium/api.md at master · tightenco/symposium · GitHub
[14:03:18]  <elazar>	elstamey: I'm thinking the plugin could have an adapter interface and take one or an array of multiple adapter implementations via configuration, so it would the potential to aggregate data from multiple sources.
[14:04:13]  <elstamey>	that sounds reasonable
[14:11:36]  <elstamey>	i'm not sure how to implement that within phergie, but I look forward to learning it
[14:14:17]  <PSchwisow>	elstamey: an example of the pattern: https://github.com/PSchwisow/phergie-irc-plugin-react-url-shorten
[14:14:19]  <Phergie>	[http://gsc.io/u/43] PSchwisow/phergie-irc-plugin-react-url-shorten · GitHub
[14:14:40]  <elstamey>	thank you!
[14:15:36]  <elstamey>	:)  You all are so helpful!
[14:15:53]  <PSchwisow>	The way you do configuration on the adapters is similar to what I found on some of the other plugins for formatters and the like
[14:16:35]  <PSchwisow>	come to think of it, the formatter pattern and the adapter pattern are actually fairly similar
[14:16:55]  <elstamey>	:)
[14:24:51] 	chrismeller (~chrismell@habari/cabal/chrismeller) left IRC. (Ping timeout: 255 seconds)
[15:06:01] 	gelignite (~gelignite@mue-88-130-117-144.dsl.tropolys.de) joined the channel.
[15:42:01]  <elazar>	elstamey: People typically want to know the conference the CfP is for, a URL to submit talks to, the closing date/time of the CfP, and sometimes the opening date, though that's more rare.
[15:42:34]  <elstamey>	OK.  That seems reasonable
[15:42:36]  <elazar>	So, I'd tend to suggest having a value object class that includes the first three attributes, and then have your adapters deal with arrays or some other type of collection containing instances of that value object class.


Tu devrais faire des rebases de ta branche `develop` par rapport à la mienne plutôt que des pull...


There were no reported bugs on it, but I could deal with the null items in collection just in case.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 739](https://teamcity.labkey.org/viewLog.html?buildId=275382&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 13 (2 new), passed: 229, muted: 23 Build time: 00:34:16
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.ScrumtimeTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.JUnitTest$RemoteTest.org.labkey.core.admin.test.SchemaXMLTestCase: <no details avaliable>

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

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

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

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

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

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

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

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

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

##### there are 3 more failed tests, see build details
```


Great. Thanks alot! :+1:


don't need square brackets around `\d` (it's effectively one character)


It's not the same!


With the change, we can't have multiple expectations in one test case.
I don't think we need to support special symbol in description because this class is only used in our unit test.


I tried... no luck.  I just get 0xFF for temp when I do that.


yes, I left this for backwards compatibility, someone might have written a custom sql query factory.


I did a quick grep for 𝖆..𝖟 and found the following also:

frege/prelude/PreludeArrays.fr
frege/prelude/Math.fr
frege/lib/PP.fr
frege/data/TreeMap.fr
frege/data/MicroParsec.fr
frege/data/JSON.fr
frege/test/QuickCheckArbitrary.fr
frege/compiler/common/Types.fr
frege/compiler/common/Binders.fr
frege/compiler/common/CompilerOptions.fr
frege/compiler/types/Global.fr
frege/ide/Utilities.fr
frege/tools/MakeDocIndex.fr
frege/control/monad/State.fr
frege/java/util/Jar.fr

2015-10-01 21:36 GMT+02:00 Dierk König <notifications@github.com>:

> Sure.
>
> Dierk
>
> > Am 01.10.2015 um 21:27 schrieb Ingo Wechsung <notifications@github.com>:
> >
> > I guess you did this with a program. Can you do the same to JSON.fr?
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Frege/frege/commit/a7dfe5a6f7397fc1a380ecaf9489ef91f8d10659#commitcomment-13549339>
> .
>


How do you distinguish between someone who is in jail and someone who is "just visiting"?


@valib I don't know about this, because we design UMS to be a certain size. I like that it resets to the default size when you restart the program. I'd be more in favor of doing the opposite and disabling resizing of the program.


Ok, you're good to go!

Charlie Rosenbury
417.693.2229

On Thu, Oct 1, 2015 at 1:39 PM, Charlie Rosenbury <rosenbury@gmail.com>
wrote:

> Yep I'll get it published when I get back to the office
>
>
> On Thursday, October 1, 2015, Daniel Hwang <notifications@github.com>
> wrote:
>
>> Is there anything I can do to help release this fix to npm?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/chuckmo/grunt-ssh/commit/f5ce3623f4e0d4cd6f80ba56077c87345e8c641c#commitcomment-13545924>
>> .
>>
>
>
> --
> Sent from Gmail Mobile
>


This sprite method was removed sometime between Phaser 2.2.2 and 2.3.0.  The upgrade caused the break.


No, it sets the default.
Later when function is called (if there are parameters) it gets overridden.


I also meant the naming of `connection` -> `connprovider`.
But yes, that as well.


Damn, that's frustrating. Ok, next question: why `setTimeout`?


This returns an empty array by editing, exporting theme.


Consider not using floating points to represent amounts of currency. `decimal` is typically used in .NET. (Note that Monopoly does not typically deal in non-integral amounts of money, but power to ya.)


lg


IT LIVES!


- [ ] again, this needs to derive from `dht.Context()`


Yay improved requirements


Yep, as expected :) Nice, just double checking.


This causes evaluation problems now on linux where `stdenv` doesn't contain `libc` attribute. It has `stdenv.cc.libc`, but this line looks darwin-only anyway and it's applied on all systems.


TeamCity pull requests :: netty [Build 357](https://garage.netty.io/teamcity/viewLog.html?buildId=1946&buildTypeId=pulls_netty&guest=1) is now running


Sorry I don't know why I've not noticed that's a function.


nvm, ignore that wrong line.


You right, I've attached tries reset to the wrong line.
It should be at line 98, before the while.


cool. this will actually eliminate the freezing issue users wid 1gb ram are facing. was about to suggest ;)


Hm. Why "-1"? Related question: why is the closing bracket indented by one in pep8 style but is not in google and chromium?

I think in pep8 mode yapf should either indent it by 4 spaces or not at all (compared to the opening bracket).


Does this compile? The name of the parameter doesn't match the name where it is used :tongue:
So the parameter should be camel-cased as well as in:
```private static int animals(boolean yesWellHeresTheThing)```


That's ok, github's diff is not the best visualization tool ;-)


return false?


I don't think this interface qualifies as what is normally referred to as a "deque". It *does* however, look very similar to an `IEnumerable<IPlayer>`, plus counting and shuffling. Consider refactoring the shuffling into a separate routine (since the players don't need to be shuffled more than once), and just using `IEnumerable<IPlayer>` (which .NET has a `.Count()` extension method for).


:+1: thanks @talvo!


I've committed a new version of winner() function to make it more readable. Please take a look. Thanks.


This names had changed and there is new ones needed to make PWM work on newer kernel versions, maybe is better add a warning about it on the commit message.


@SubJunk Disabling resizing would be a mistake, people have many different screen size and resolutions and there isn't one solution that fits all.

I think saving the size and position is a good idea, at least under Windows since that's "standard behaviour" - i.e what users would expect. I don't know what is standard on the other platforms. Remember that many have multiple screens with different resolutions, it's a pain with applications that you have to place correctly every time you launch them imo.

As long as this (I haven't checked) doesn't change the default size, any saved changes would be because the user wanted this. Much of the UI doesn't scale well, but I'd rather consider that as a problem then trying to enforce the size.


How to use:

**default:**
`<img image="{{image.url}}"  />`

**with custom loading placeholder**
`<img image="{{image.url}}"  image-loading="/image/loading.gif" />`

**with custom fallback placeholder**
`<img image="{{image.url}}"  image-holder="/image/error.png" />`

**with custom fallback and loading  placeholder**
`<img image="{{image.url}}" image-loading="/image/loading.gif"  image-holder="/image/error.png" />`


test


We could always cache a copy of the config. It should never be large.


same as above


eh, more or less, been to busy to do any real work on SOS x.x


You wrote the same in if and else part :) main is independent of opts.prerender


Nope, it's a verb!


This completes the basic unidirectional, yet linear, process flow of Flux. Usually, though, the unidirectional process has a cyclical flow and it doesn't necessarily end. The following diagram illustrates a more common flow. It is the same idea again, but with the addition of a returning cycle. Eventually the components depending our our Store data become refreshed through this looping process.


looks good


@ryanrousseau my bad :-1:


If I remember correctly, I think that `column` is 1-based. I might be wrong on that though. As for the indentation, it's just how Google/Chromium states in their style guide. I'm not sure about indenting it 4 spaces compared to the opening bracket. It doesn't seem to be the PEP8 way (I made the change intentionally based on a bug where we were violating a PEP8 dictum).


<elazar>	And https://github.com/phergie/phergie-irc-plugin-react-command for parsing commands.
[15:45:30]  <Phergie>	[http://gsc.io/u/50] phergie/phergie-irc-plugin-react-command · GitHub
[15:46:27]  <elazar>	Something like:
[15:46:31]  <elazar>	!cfp to get the conference with the closest ending CfP date
[15:46:38]  <elazar>	!cfp [search] to get data for a specific conference for which the name contains the search phrase
[15:46:57]  <elazar>	And maybe !cfp all that returns a PM to the requesting user with a list of all active CfPs. (Don't want to spam a channel with that.)
[15:47:30]  <elazar>	Or possibly !cfp # where # is an integer to get the next # conferences relative to ending CfP date with a reasonable (maybe configurable) cap.
[15:47:51]  <elazar>	Also possibly !cfp random to get a completely random CfP.
[15:49:45]  <elazar>	^ joepferguson: Idea: a phergie-irc-plugin repo where plugin ideas can be tracked, discussed, and specced out like so.
[15:50:10]  <joepferguson>	hrm
[15:50:12]  <elazar>	Basically to better manage https://github.com/phergie/phergie.org/blob/master/source/plugins.md#ideas-for-new-plugins
[15:50:12]  <joepferguson>	yeah.
[15:50:12]  <Phergie>	[http://gsc.io/u/31] phergie.org/plugins.md at master · phergie/phergie.org · GitHub
[15:50:39]  <elazar>	Would allow veteran contributors to better mentor new contributors I think.
[15:51:21]  <elazar>	Could possibly also house a guide to the ecosystem (e.g. commonly used plugins, patterns, etc.).


No need, the return below will take care of this.

(Also, this changed in the current version to support multiple links from one single discovery response packet.)


Meh, old habits, it's how nearly every one of the Mock objects was spelled at my old place of work.... old habits die hard.


Please do not comment out code, just remove it.


probably best to implement actual wrapping/wordbreaking here instead of just increasing the width of the popup


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.5.64.3b41076b089d959d7fbd1e5e2ba7178ef88541e0](http://build.palaso.org/viewLog.html?buildId=63981&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 874, ignored: 50 Build time: 00:33:51


Will return EINVAL here.


the way you have this is fine & functional. fyi, alternatively, you could do something like
`re.sub(r"\\([\"'])+", r"\1", text)`
which will find `\\([\"'])+` (backslash followed by either a double or single quote) and replace it with `([\"'])` (the first group, which will be either a single or a double quote)


i'm not sure that this helps much, i would just have users always pass main queue.


You should do that on the master branch as well


But now you are missing R6 ...


Yeah, the best solution would be a hashcode but for while I think that's sufficient.
In `XWebViewTests` is impossible to expect `127.0.0.1` so I've done that. The test for `loadHTMLString:baseURL:` when `baseURL` is a local folder need to check that a local server was created.


@taconaut thanks, to spoof the installer I had to
* ```mklink /d ums ums-mlx``` in ```%programdata%```, first making sure that *ums-mlx\debug.log* existed.
* I also had to set ```code_enable=false``` in UMS.conf to get around an NPE for jumpy to load (will look into it, separate issue).

MLX still behaves as before vis jumpy: loaded but no content showing and otherwise seemingly normal. For comparison I tried Channels too and (perversely :-) got duplicate 'Channels' folders showing in the xmb root instead. Don't know if it's significant here, but jumpy implements ```AdditionalFoldersAtRoot``` vs Channels ```AdditionalFolderAtRoot```.


For read-only case, I am not doing anything. For read-write case though, I am removing (well, was removing -- in latest version it's just emptying) the "variable" (a file containing a string) named uuid-for-children. Let me explain why.

In general, every time I need to clone a ploop, I need to make a snapshot in parent ploop and use it as a base for new ploop. Now, if I need to make more than one clone, and parent has not changed, I can reuse the same snapshot, and this is what I'm doing by saving snapshot UUID to a "variable" (a file) named "uuid-for-children", and checking if such "variable" exists next time I'm doing clone. This is an optimization that saves number of deltas in ploop (which is limited to 128), as well as some time. I initially implemented similar optimization for Docker driver, as in Docker there is a lot of cloning going on.

Now, if I cloned something from a parent, and then the parent is mounted read-write, I can no longer rely on the snapshot saved to uuid-for-children to contain the latest data. Therefore, I remove the contents of uuid-for-children to mark that I will need to create another snapshot if I'm to clone this ploop.

I know it all sounds complicated. The big problem, of course, is I can't ask ploop "if there's something changed since this particular snapshot or not?" (maybe including or excluding filesystem metadata changes such as files mtime or filesystem mount count etc). This is what needs to be implemented (although I don't know how -- maybe you can help), and once it's there we no longer need such invalidation.

There's yet another fix to be implemented -- that is, if I am cloning a ploop which is currently mounted read-write, I can't rely on the snapshot I made being up-to-date, so in this case (cloning a rw mounted ploop) there is no need to save uuid-for-children. Again, with the ability to ask "is something changed since this snapshot?" this won't be needed.


issue5221_memoryAllocation (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


funciono


Shouldn't this be referring to the name of the repository like it was before?


Should only need the latter check there anyway so I changed it to remove the left part. Thanks!

btw, I'm @cbuechler here.


Notes how to get it building:
1. Get pkgsrc (and unpack it to to `/usr/pkgsrc`) - the best try is 2015q3
2. Get `suse*_libgc`
NetBSD 6.x: there is needed `emulators/suse121_libgc`
NetBSD 7.0: there is needed `emulators/suse131_libgc`
(for 32-bit platform there are `*_32_libgc` packages)
These packages are in current and shall be obtained from
3. Get pkgsrc-wip (pkgsrc.org/wip)
4. `cd /usr/pkgsrc/wip/opendylan && make install`


Hmm, they might be unconventional but you know, `.+Short` should be short I suppose? :baby:


Leave a comment/etc/gitconfig  - - system  .config/git)


Yes, fixed it now


I don't like exposing this property. We specifically didn't want to expose it before, because with typical usage of this through the OpenMRS API you will end up with a possibly-stale value.

I would much prefer to remove the changes that expose this property, and instead have a separate REST endpoint that gets and sets this via direct SQL queries...


weeeena jaja


The AssemblyInfo.cs file appears to be missing.


cacha


Player is not starting again after hitting fireball


Flashing the modem partition with stock firmware helped. This information might help some others. I haven't even mounted it rw until after the problems started occuring. Thank you for your assistance.


Awesome thanks.

> On Sep 30, 2015, at 6:50 PM, Austin Cooley <notifications@github.com> wrote:
>
> take a look at willBeginBatchFetchWithContext
>
> —
> Reply to this email directly or view it on GitHub.
>


make compiler happy


should this `return false` instead?


PEP8 standard violation:Many spaces


what do you mean a corresponding property?  it can be used alone to determine if the element is user editable.
https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/contenteditable


For inside a for,While inside while  more time taking to load game


Excellent run at STC...
It's amazing that this was never tested !
But why not [0,4] for this test ?


jump not implemented


Thanks...because I made a bunch of code changes.
https://github.com/VoyagerOne/Stockfish/compare/77b4f4c...d7ad797


It's per file: https://github.com/BIC-MNI/libminc/blob/develop/libsrc/hdf_convenience.c#L2212


when player drops down on a coin or a fireball, it gets struck


Long method code smell


polymorphism not used


security through obscurity,

Guess I should revoke that Client ID, and Client Secret pretty soon.

Was going to use cocoa pods keychain, https://github.com/orta/cocoapods-keys <https://github.com/orta/cocoapods-keys>, then just forgot.

YOLO

> On Sep 30, 2015, at 6:49 PM, Austin Cooley <notifications@github.com> wrote:
>
> LOL i'm pretty sure your account is definitely blocked by instagram by now :)
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/regnerjr/AsyncInsta/commit/469db351304b060fb4a4e3aed2a27b7f94c8e4f5#commitcomment-13532191>.
>


Ok, I added a Take() to the Where and the tests passed. I went to do a pull
request on GitHub and I think it created the request but something was
different and I didn't feel confident it went through.

On Tue, Sep 29, 2015 at 11:21 PM, Michael L Perry <notifications@github.com>
wrote:

> This line is where you should be getting your out of memory exception.
> There's a simple change you can make right here that will make the problem
> go away. Notice that the test itself includes a Take(5). That will prevent
> an infinite loop or out of memory exception if you don't force it to
> allocate memory and finish the loop right here.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ramichaels/eval/commit/ffd1983e178d510d9f590821f4a15b5eb3abcdd9#commitcomment-13508581>
> .
>


Ah ah...I thought it was only the *3 that was removed !


If that was the case...the test would have failed quickly.


Hi @krieger-od, it's true that it is not explicitly documented in the CodingStyle but it is widely used though. I don't know how to override that tbh... I'll try to dig this out.


Hey @jongerrish,

I agree that this is an issue, but it doesn't appear to be a regression. I checked out d6b681b (the revision on master immediately prior to this PR being merged) and ran your test (I had to leave out the line with the `RuntimeEnvironment.isMainThread()` test, because that function was only added in this PR). The test failed - `myLooper()` does indeed return the main looper, whereas in real Android, the call to `myLooper()` should return `null` (because the background thread(s) won't have loopers associated with them).

This accords with my understanding - ie, that Robolectric has always run everything on a single thread, even the "multithreaded" stuff. This is certainly the case with `ShadowLooper` (I raised this issue in point 5 of #1879), and this indicates that it is also the case with `ShadowAsyncTask`. As I noted in #1879, this has the potential to confuse code running on background threads as certain expectations will be violated (eg, getting non-`null` for `myLooper()` or getting the main looper when you expect your own looper).

I intended to look at this (among other issues) at some point soon anyway while cleaning up other scheduling issues, so if this is a pressing issue for you I will prioritise this one instead.


Hi, this fix makes jquery-ui as a dependency for $(origCell).zIndex().
Maybe consider $(origCell).css('zIndex')?


os.Open open file read only, so WriteString won't work.


@vangheem Which tool is actually influenced by this `noqa` comment? It has no effect for pyflakes or pep8, at least not the way I have set it up.

When an import like this is not used, which pyflakes complains about, but we still want to keep it, I always add a line that simply mentions the import. And I add a comment `# pyflakes` at this new line, which has no influence on any tools, but only serves as an explanation for developers on why this line is there.  So:

```
from log import log_exc
log_exc  # pyflakes
```

I have also seen the following in other files:

```
from my.package import unused_import  # nopep8
```

This also has no effect on tools that I know.  If pep8 listens to this, then that is nice, but pyflakes will still warn that the import is unused.

I am just wondering if I am missing some other tool, or a different way of setting things up.


should use  os.Open to create one file handler for read, and os.Create to have another to write.


Cool! Very nice!


Your 'if' should be lowercase.


issue5221_memoryAllocation (rongpengzhang) - Win64-Windows-7-VisualStudio-12: OK (1896 of 1896 tests passed)

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


This should have tests :(


"Santiago de Jesús Rodríguez Córdova"


issue5221_memoryAllocation (rongpengzhang) - i386-Windows-7-VisualStudio-12: OK (1896 of 1896 tests passed)

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


```
~/-tg-station/code$ git grep "^obj"| wc -l
189
```


Steve,
This and password reset are the last part of my basic authentication system.  I am stuck on the sign_up_spec.rb test.

```
click_email_link_matching(/#{edit_account_activation_url(activation_token: wendy.activation_token, email: wendy.email, id: wendy.id)}/)
```
I installed the email_spec gem which gave me a lot of helpers.  This click_email_link_matching expects a regex and it would not match /Activate/ which surprised me.  Here is the error:

```
     Failure/Error: click_email_link_matching(/#{edit_account_activation_url(activation_token: wendy.activation_token, email: wendy.email, id: wendy.id)}/)
     RuntimeError:
       No link found matching /http:\/\/www.example.com\/account_activations\/210\/edit?email=wendy_ricketts%40icloud.com/ in <html>
         <body>
           <h1>Core Athletes</h1>

       <p>Hi Wendy Ricketts,</p>

       <p>
         Welcome to Core Athletes!!  Click on the link below to activate your account:
       </p>

       <a href="http://www.example.com/account_activations/A_62q1Db3JUNug00VqtGXQ/edit?email=wendy_ricketts%40icloud.com">Activate</a>

         </body>
       </html>
```

Somehow I cannot get the program to give up the activation_token.  It is also not working in real life.  If I past in the link from the server log, it gives the error "Invalid activation link".  Everything matches except the randomly generated token.  I am going to create a new branch and start styling, the site is in bad need of some clean up now.  I might return to this after some styling.


Get rid of .DS_Store from git


'elif' is what you use for extra 'if' cases if you have more than two (an if and an else).  The 'elif' cases go after the 'if' and before the 'else'.


Remove this.


so this is going to break bluetooth again also does this fix the prox and light sensor issue? I've been trying to find the cause but still not sure.


iterate all items into one funciton


Thanks for reply.
Another note: adding ",M1" makes closing parenthesis (when first on the line) NOT to align on opening parenthesis, but to some tabulation level. I prefer the latter.


order


@property (nonatomic, weak) UISearchBar *searchBar;


issue5221_memoryAllocation (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (822 of 822 tests passed)

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


Should this be called getParentMount?


return [self.results count] > 0 ?: 1;


@Gamecrafter If you want you can contact me @ MajikalPayton on KIK, I don't have any particular social media site I am active on right now :/


Long method, a method, function, or procedure that has grown too large.


>  it's true that it is not explicitly documented in the CodingStyle but it is widely used though

If anybody cares, I dislike alignment on open paren. also because it is usually too close to the 80th position...


Would it make sense to also add a test checking that the function is a generator. Something like this: http://stackoverflow.com/a/6416855


Maybe invert this check and move the Mount call to make private inside here?


Just like comment above, does it make sense to replace "Source" by "Parent" in this function name?


It's better to have #define placed at the top of the file after import statements.


s/relation ship/relationship


working on https://github.com/datacats/datacats/issues/334


nice


@MajikalPayton Hmmmm.. Kik, you say. I can't, I don't have a smartphone. :sob: What about Twitter?


Refer to GCD.


Good point about keeping version history too.

> I would leave it as is for now and if we know that we won't introduce any changes in filename or folder names with v1.0 we can update the Readme accordingly.

We could go the opposite way: suggest a simple rename of the directory for now, and if we end up making changes to the filenames then come up with the recommended way to upgrade.  Or more than one way, such as: either use `autocreate_reference_screenshots`, or rename the individual files using a command like

```
$ find reference_screenshots -name "*.png" -print | ruby -rpathname -n -e 'Pathname.new($_).tap { |p| system "git", "mv", p, p.sub_ext("-1024x768.png") }'
```


Better to have the base url defined in a .plist file.


i do not know why?
case one===>
autocmd Filetype java set onimifunc=javacomplete#Complete
autocmd Filetype jsp set onimifunc=javacomplete#Complete
filetype plugin indent on
java file onimfunc is javacompelte
jsp file is htmlcomplete
but
case two ===>
filetype plugin indent on
autocmd Filetype java set onimifunc=javacomplete#Complete
autocmd Filetype jsp set onimifunc=javacomplete#Complete

all of java jsp file onimifunc is javacomplette


Be more specific.


Please make your own list for this assignment.  Also, name it something other than 'list' because 'list' is a reserved keyword in Python (it tells Python to do something specific, like 'elif' or 'def')


Thanks.


[errors count] > 0


Rather than switching themes, I'd rather Picard just not be able to display any content (with the helpful notice in the admin). Sort of a weird encouragement to actually install the plugin, than just making do with the default theme that's now active and ready to go.

The notice could also include a direct link to install the plugin.


Add more #pragma marks


Now it's ready. Pull! :-)


Better to have the first enum element set to a certain integer value.


Excessively short identifiers, cope smell


Hey Oxyrus, i had a similar problem, so to fix it i ran the following command:
$ meteor add ejson .

That fix my problem. Here i found that solution: https://forums.meteor.com/t/iron-router-not-working-with-meteor-1-2/10196/6
:+1:


這個寫法可以直接下 join tables 的 where 查詢


self.results = [NSMutable new];


Remove this.


@paked No worries!


Not so. Knuth has a bounty for corrections and he has paid for a great many. I have only the second edition and Volume 2 by itself has 130+ pages of errata!


Need to remove the `,` from the end of this line, borked my webpack.


Excessive short identifier


See [lodash.bindall](https://www.npmjs.com/package/lodash.bindall) and [others](https://www.npmjs.com/browse/keyword/lodash-modularized).


Uhh... I'll get on twitter for a bit.


Downcasting, code smell


header {
要素とカッコの間にスペースを入れましょう！


Excessive use of literals


Okay, my user is "The_Gamecrafter". I think I've seen you on there, but I forgot.


No worries. Just commit so we can all review. I'm learning and re-learning a lot of things myself.


せっかくsassなので
```
margin {
  left: 20px;
  right: 20px;
}
```
的な感じで書きましょう


ちょっと直してみてください！


queue can't be null here because of the check, ?: seems unnecessary


Only requirement for this release was that landing on "Go To Jail" teleports you to jail, I believe the implementation of jail comes in a later release :)


Won't fix, as that's irrelevant for the WIP version.


Why change the SdkVersion?


Again habit... I see money and think floating point haha. Then I later noticed it will never be used like that for this game haha. I'll change it though


Comment...


Thanks! Updated to your version. :+1:


This should fix issue #4 and issue #65.


> So it would be something like this.

Yup, exactly.

> Do you mean to not have the option at all, and always pass routeData object to handlers? And we won't need to worry about backward compatibility?

Yeah, and push it to v2. However, if you can make it work behind a flag in a backwards-compatible way as you have here, that would definitely be smarter. We can always flip the switch or remove the flag for v2.


Yeah, for some reason my mind mixed up the idea of Deque with what is really a Queue. Just wanted to have the shuffle method in here as well. Perhaps I'll make this just a generic Queue with a shuffle method, such that I can shuffle actual "Deck"s of cards for Community chest and Chance cards


would love a tagged release for this when you get a chance


I was imprecise in my phrasing. By "always goes at the end" I mean always after the named parameters, the last match. You can have multiple splats, but there is never an interleaving between named params and splats, so concating the two match lists should likely be fine.


I like this change. Going with the historical CAPE is a good move.


instead of passing in a data and a starting offset, you should slice the data object in the caller and just pass in the subrange.


Ok, vu


@Enriikke would you include this logic here, in the view?


this can be released much earlier on 64

also, any reason why this fine can't be arc?


Hmm, this is the kind of stuff that goes well with a functional test. Anyway, I understand this is v0, so let's keep that in mind.


If I remember correctly, this was necessary for the speed up to be maximal, but it was hard to design a test that would measure the number of times each optimizer was called.
I'll have a deeper look.


Enhetstest?


+Ajout test....


Uh, nice.


I also tried making the Javascript do some more things, but I couldn't get anything to work, so the Javascript is back to the way it was before.


I prefer writing this as `switch` or `== && == && ==`. Using `ImmutableSet` feels too much of roundabout to me.


Moving to F# 4.0 is fine - the question is if the library itself needs to target FSharp.Core 4.3.1.0 or 4.4.0.0.  Clients apps using the library can of course bind to any higher version and redirect

If the library is using attributes to enable the implicit quotations then yes, you will need FSharp.Core 4.4.0.0, so you can ignore this, thanks!


OK


great!


Robolectric doesn't support SDK v22 (21 is original Lollipop, 22 and 23 are out but not widely used). First attempt to fix broken tests was to change project to use 21. Second pass (what's in the final PR I think?) is to use 22 for the project but insert a bunch of `sdk =21` args into the test runners so Robolectric doesn't throw.


ok, makes sense!


`only list them and log the parameters they take`?


Pour la compilation des assets, en commentant cette ligne et en effaçant les assets, la compilation passe sur heroku ! :+1:


Just importing it doesn't cause it to be exported...

If you want a flat export, change this to:
```
export { ICardContainerService } from './cardContainer.service';
```
or
```
export * from './cardContainer.service';
```


Is there any reason to both buffer the output and return it in an array, and also send break output lines and call callbacks on them?

Commonly, stream processing libraries splits these up:
1. something which reads chunks and then calls callbacks on the chunks
2. something which takes chunks and calls callbacks on the joined lines
3. something which takes chunks and buffers them, and returns the full buffers at the end.


See https://github.com/Theano/Theano/pull/3092#issuecomment-120208843
There were more passes in "canonicalize" without that change.
Maybe this is not the case anymore with the current PR, but we should check.


This is where I restored the old versions (22) before submitting PR.


Cheers, I'll try to set it up when I'm at the pc.

Jumpy won't show up automatically, you have to specify where it has to show up. For this, go to media library -> tree then right click on a folder -> add and jumpy should be in the list of folders that can be added.


test


And this is an example of the (many!) extra args that get Robolectric to still work.


Yes. Line 75 generates the HTML `select` element and line 76 (and following) generates all `option`s for the select element. Line 80 makes all options selected and line 82 hides the whole HTML element.
That way, the corresponding controller knows which `EvidenceItem`s to save for the `SearchResult`.


why do you need to add 1?
Also, I'm assuming that if you don't supply `decimals`, then `decimals + 1 == NaN`...but it's not terribly clear. Maybe because I'm not a JavaScript programmer.

Finally, the old implementation used `Math.log(bytes) + 1` where you just use `Math.log(bytes)`. Is this still correct?


perhaps it's better to just change the callers instead of wrapping this in yet another block.


What if we simply had by default another results output dir for dry run tests instead of forcing a job id to them?


That'd be awesome, whenever you get a chance (I'm swamped these days too). It makes a lot of sense as a feature.


Why `*y`? Is it supposed to be a list or 1?


I can understand your concern. But description of expectation is defined by us, so we can avoid the special case, for example:
127.0.0.1 ==> localhost
www.github.com ==> www_github_com (or just github)

I know these may look somewhat ugly, but it's understandable, and more important, make unit test code simple.
We know the limitation, so we can treat the description as identifier.


si


@wlipa Seems necessary. At least - it was necessary for me.


Fixed


@philwebb, what do you think about merging your entire `week-2` branch back onto `master`?


I don't think this change was intentional, probably a result of the merge. Reverted.


@taconaut jumpy doesn't appear under 'folder -> add' (but Channels does). Both plugins show up in the plugins tab.


Thanks David. I tested and it is working the same. Attributes assign true. Associations also assign true in these cases so that they can never fail. Not sure why you saw 'removed' OCL The child class should have the same method name of the parent class (now overridden) and the OCL in the operations associated with it should be 'true' (and removed). The only place this differs is for custom constraints. Since we can define the OCL ourselves for them, the severity can be left set (preferably SHALL) and the OCL manually written as 'true'.


What do you mean, "later optimizations"? What optimizations are missing in `self.mode`?


I prefer writing this as `switch` or `== && == && ==`. Using `ImmutableSet` feels too much of roundabout to me. I understand performance is not a concern here, but I don't like the roundabout. I searched for `ImmutableSet\.of(.*)\.contains` in Presto and the only existing occurrence is in `MockHiveMetastoreClient` which is full of code style violations.
Ditto for the many other places of `ImmutableSet\.of(.*)\.contains`.


Don't need to do it this way.  I based the pattern off of the BMP180 Driver which has the same toggle mode.  If you think there is a better pattern here, let's do it.


We should only send this if it's not nil, because sending a nil value might make SOAP choke :)


WTF cleanup performed XD

On 1 October 2015 at 21:31, KostyaBondar <notifications@github.com> wrote:

> Maybe some service would be better for this ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RobbyWD/ireland/commit/9afe0148d8291c1cab8989dd5480c0fd9c9924ab#commitcomment-13547984>
> .
>


The syntax was also formatted inconsistently (cosmetic).


See other comment. Nothing will be auto exported unless you use `export * from './some/file';` and this will only auto export all of the explicit exports in a child module.


I've added a tint change handler in cf4f32975943e03485fe4dd7454190902b3762c5, but it would be nice if someone could review this.


Also, there's an extraneous dot at the end of the argument parsing, a syntax error.

    diff --git a/avocado/core/plugins/runner.py b/avocado/core/plugins/runner.py
    index ac62415..f11039f 100644
    --- a/avocado/core/plugins/runner.py
    +++ b/avocado/core/plugins/runner.py
    @@ -51,8 +51,8 @@ class TestRunner(plugin.Plugin):
                                      help='List of test IDs (aliases or paths)')

             self.parser.add_argument("-d", "--dry-run", action="store_true",
    -                                 help="Instead of running the test only "
    -                                 "explore them and log their params".)
    +                                 help=("Instead of running the test only "
    +                                       "list them and log their params"))

             self.parser.add_argument('-z', '--archive', action='store_true', default=False,
                                      help='Archive (ZIP) files generated by tests')


On this very cset, with a forced fail..:
```
======================================================================
FAIL: relengapi.tests.test_db.test_ensure_empty
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/callek/mozilla/obj/relengapi/local/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/home/callek/mozilla/git/build-relengapi/relengapi/lib/testing/context.py", line 133, in __call__
    wrapped(*given_args, **kwargs)
  File "/home/callek/mozilla/git/build-relengapi/relengapi/tests/test_db.py", line 72, in test_ensure_empty
    eq_(1, 0)
AssertionError: 1 != 0
-------------------- >> begin captured logging << --------------------
relengapi.blueprints.slaveloan.bugzilla: WARNING: No bugzilla user specified. (Set BUGZILLA_USER in config to fix) (user='anonymous:' task_id='ae2a87dc-63bc-464d-867a-a91f2ca0aab7' request_id='91f86c96-1a9d-4e88-b965-371fdb990e9a' archiver_task_state='STARTED' task_name='relengapi.blueprints.badpenny.execution._run_job' archiver_task=u'foo')
relengapi.blueprints.slaveloan.bugzilla: WARNING: No bugzilla password specified. (Set BUGZILLA_PASS in config to fix) (user='anonymous:' task_id='ae2a87dc-63bc-464d-867a-a91f2ca0aab7' request_id='91f86c96-1a9d-4e88-b965-371fdb990e9a' archiver_task_state='STARTED' task_name='relengapi.blueprints.badpenny.execution._run_job' archiver_task=u'foo')
--------------------- >> end captured logging << ---------------------
```


Shouldn't this be one in 25!/(25-5)! chance ---- Basic permutation 1/6,375,600

Assumptions: One ACME validator and assuming that the attacker can get out a new random certificate in between every request.


do you want me to use Option and flatMap? :)


Oh, there you are.


If you don't have sensors, why try to display the temperature in your prompt?
The right thing to do is to set `LP_ENABLE_TEMP=0` in your `~/.liquidpromptrc`.


@skeptical Then it needs some more debugging :) it seems I've missed something with AdditionalFoldersAtRoot (I've only tested AdditionalFolderAtRoot)


OK, I just realized the dry run results get to a `/tmp` dir, so disregard the last comment. The question is, now why force the job ids? just to reinforce they are dry run results?


just a question... won't that break existing addons that rely on 'Roles' being given as a form value?

Also in line 462:
```
$Roles = $this->Form->getFormValue('Roles', null);
```
How about moving 'Roles' from formValues to Data?
```
// Here are the initial data array values. that can be set by a plugin.
$Data = array('Roles' => '', ...);
```


This should only crash when using cudnn v1, right?


@igalic sorry, the spec file stays as it was (setting the collect_real_version fact), the template must use collectd version as you've reverted here.

That should make the tests pass correctly (at least for Puppet 3.x)


@jherico Is it ok to remove this?


Ah, ok that is fair.


Would be more clear to separate out the code and prose on new lines.


You should detail in a comment why this is necessary, as this basically subverts the use of `poll`. Doing it like this can cause `read` to block, which prevents the other file descriptors from being read, which may lead to deadlocks.

poll will always set `POLLIN` when there's data to be read, therefore you should never encounter the case when there's data to be read and `POLLIN` is not set when you call poll.


It could also be bad behaviour from jumpy (due to hacks :).  Interesting that Channels automatically shows up in duplicate without even being added.


@phlax I see you added ``*_removed`` in a later patch.  That I think covers it.  Summary seems to be arrived, removed or changed.

I find the ``_new`` and ``_added`` distinction slightly confusing.  I guess that relates to the fetch and pull distinction also.  It might be more clear to have it illustrated somehow.  Don't know for sure.

This feels like it is shaping up nicely.


-accplot.m runs a script to plot data from eps.dat (Hf-W tracking for one body)
-makeinput.m is a GUI script to generate accrete4.inp, the input for accrete5e


@JesseWeinstein, I'm fine with this. Let's remove them everywhere, though (I can do this as well, but if you'd like to add that to this PR, that would be welcome, too.).


I also tracked down the source of those:
```
[WARNING] [10/01 13:16:14] QQmlComponent: Created graphical object was not placed in the graphics scene.
...
[WARNING] [10/01 13:16:14] QQmlComponent: Created graphical object was not placed in the graphics scene.
```
to
```
    function addMenu(menu, newMenu) {
        return menu.addMenu(newMenu);
    }
```
in VrMenu.qml line 190 but I could not figure out what the code path was beyond this point.


i think this breaks stuff


Missing a closing bracket.


Does `jspm` really need sudo?


When does this happen?

It seems like that function returning nil should be treated as an error, not silently ignored.


Can we add some tests that assert it doesn't "find" values that happen prior to subscription. (It sounds silly I know, but it's an assertion)... Basically take any of the tests above and have the hot source start with `---a---^---b--...` so that way there is one event on the hot observable prior to subscription. You and I know there's no way that should effect anything, but who knows what sort of changes the future holds?


When adjusting the space between the h3(or any heading tag for that matter) and the elements around it adjusting the h3 elements margin/padding is the best practice. I can see in the embedded css that you have set the margin-bottom of h3 to 0px and that should accomplish your goal, and would be the standard way to address this issue. However you are missing a closing } in your css for the footer selector so the last 2 declaration blocks aren't being processed by the CSS. If you add that  } in you'll see your html being styled correctly so that the h3 tag no longer has space between it and the next element (Unless the element next to it also has a margin or padding se)t.

Additionally I'd suggest a couple items here. In this instance the content in this h3 tag has 3 distinct content categories.
"The Best Chocolate Chip Cookies" --> this is truly our h3 element. It's the title of the recipe and should be the only content actually wrapped in the h3 tag.
The "Recipe Author" and the "Prep Time" are distinct elements and using two separate spans was a good choice, but it would serve the flow of the content more if they were outside of the h3. In HTML5 there is the address element which is used to describe contact information so that could be an option, in many cases simply assigning a css class of author to the element you are using is standard practice.


Yes, I re-added the check.


I see no reason to specify a timeout if all you do is go back into poll again.


Isn't that a bit broad?
http://www.eclipse.org/jetty/documentation/current/alpn-chapter.html#alpn-versions  list quite a bunch of versions for JDK7 and JDK8 depending on the build number.


fyi, alternatives:

You're working with `match_obj.group()` which gives you the full matching text, something like `%1$s`

`match_obj.group(1)` will get the first parenthetical group, which will be something like `$1` (since the first group is `([\d]+\$)`). It'll be `None` if the given string was just `%s`. So you could call `group(1)` instead of `group()` and then replace the if block with something like `new_index = int(text[-1]) - 1 if text else 0`

Or, slight variation, you could add another parenthetical group to the regex, changing it from `r'%([\d]+\$)?s'` to r'%(([\d]+)\$)?s'. Now you have a group (match_obj.`group(2)`) that's the number alone, no $. You'd still have to cast it to an int before subtracting, but wouldn't need any substring-ing.


wouldn't it make more sense to call this 'currentAppName'?


Не очень поняла тебя. Сейчас есть pymorphy для русских слов и snowball stemmer для английских. Что ты имеешь в виду под указанием двух языков?


Brought the old version back for kde4: e9fcbe67b09.


I'm sorry but what chapter are you referring to?


I would do something like this
```
values.stream()
        .map(f -> f.getSignature().getType())
        .filter(t -> t == AGGREGATE || t == APPROXIMATE_AGGREGATE)
        .mapToInt(x -> 1)
        .sum());
```


Ugh, I'm sorry, I misread the code before and misunderstood what you meant by "make it absolute".  You're joining with / to ensure the string has a leading / (i.e., it's an absolute path), then taking off the leading /, and *then* splitting.  I had thought you were splitting and then then taking off the leading path component, so my suggested change for the comment was incorrect.


Both issues are strange. Do both plugins show up twice, one as basic and as folder plugin? Any exceptions in the logs? I'd guess the second is due to directly accessing PMS.getRoot(), I'll check :)


TeamCity pull requests :: netty [Build 357](https://garage.netty.io/teamcity/viewLog.html?buildId=1946&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5616, ignored: 21 Build time: 00:37:39


I would do something like this
```
values.stream()
        .map(f -> f.getSignature().getType())
        .filter(t -> t == AGGREGATE || t == APPROXIMATE_AGGREGATE)
        .count()
```


s/fom/from/


If you want to have this code platform independent (i.e. testing on the F4 discovery ;-), I recommend using [`xpcc::stm32::ClockTree::getCpuFrequencyMHz()` from here](https://github.com/roboterclubaachen/xpcc/blob/develop/src/xpcc/architecture/platform/driver/clock/stm32/clock.hpp.in#L243-L259).


s/an folder/a folder/


Yes it was, there was a `++itr;` after that if block


Well, [a Norwegian asked this question about 11 years ago](https://www.mail-archive.com/winpcap-users@winpcap.polito.it/msg02003.html), and [one of the Italians answered](https://www.mail-archive.com/winpcap-users@winpcap.polito.it/msg02009.html), saying "I think this are reminiscences from the first versions of WinPcap, when overlapped I/O was used to interact with the driver."

There still appears to be some use of overlapped I/O in the Windows OT version of the code, but that might not actually involve I/O being done after the stack frame in question is released, and I don't think the OT code is still supported - the NT code doesn't use overlapped I/O.

So we can probably just allocate it on the stack.


Will do, greatly appreciate the heads up.

Sent from my iPhone, so just assume that auto-correct mangled my words...

> On Oct 1, 2015, at 2:29 PM, Dmitri Voronianski <notifications@github.com> wrote:
>
> @ifandelse Please take a look on your implementation once more (https://github.com/voronianski/flux-comparison/tree/master/lux) because currently your example doesn't work at all, it throws such error in console:
>
> Uncaught Error: There is no action named 'getAllProducts'
> —
> Reply to this email directly or view it on GitHub.
>


Before `el.releases` was being assigned the whole response object from the album query, which was producing the parsing issue. The assignment was also unnecessary since the site doesn't access all of the response's fields.

The build also trips up on undefined artists in the db, so I included a check there.


As an test coverage, that's definitely can be added and no harm. I'll update changes soon.


Avoid literal.


logs are normal afaict. Both plugins are listed only as basic.  I tried to tell you long ago that we plugin-writers tend to be a little free-wheeling and corner-cutting :-).


I'll update the README too, if you need/want that.


Déployé, et ça a marché ;)


Can we just introduce more variables implicitly - to contain `+=` or `-=` result and then just use it in the rule as a nested one?


А что `&` не используешь?


access token in history is already expired :)


You can update to alpha-version - I did it and everything has gone well
with xsp.

четверг, 1 октября 2015 г. пользователь germgerm написал:

> Any idea when we can expect to see this in a release version ?
> This change is in 4.2.1 latest release is 4.0.4
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mono/mono/commit/58e0608c10c621b15fd9f396f8e46c567891b231#commitcomment-13548134>
> .
>


There needs to be a different solution to this problem. The use case where this can not happen is when you have a multitouch app and you need to be able to simultaneously drag two objects through. With this "fix" the object on bottom will not move once you finger crosses into the hit region the object on top.

If you don't want the move events to propagate to other objects that are not on top, then use evt.stopPropagation();


должно быть два `__`


No worries - example hopefully makes it clearer and doesn't hurt to have in there.


could be done with one sort:
`sort {$b->[3] <=> $a->[3] || $a->[2] <=> $b->[2]} @$matches


Could you please add curly braces to the introduced `if()` and also add an `else` as it would increase the readability of the code several 100% :smile:

So, that the code would read:
```javascript
if (name === self.options.valueKey) {
  if (typeof obj[name] === 'object') {
    return self.objectToXML(child, name, namespace, xmlns, false, null, parameterTypeObject, ancXmlns);
  } else {
    return xmlEscape(obj[name]);
  }
}
```


Sorry, I missed this but we resolved it in gitter anyway (for those tuning in at home). Thanks!


Лучше писать `{'☰'}`, чтобы было сразу видно, что за символ внутри


We don't need to keep these comments any more, do we?


`switch`


Why not auto generate the file before its inclusion here? We'd have always freshly generated diagrams in the docs, which seems desirable.


The language still doesn't switch. Always Spanish to me.


I guess we want the test parameters to be listed here, *then* give the user the skip message.


Curious about the switch from **Math.max.apply** to **_.max** - isn't the former actually faster? (Basing this solely on this page: https://jsperf.com/math-min-vs-min-vs-d3-min-on-large-array/5 )


Thanks ! the fix will be in the next release.


Щито это такое?


TODO: Implement UITableView without using UITableViewController object.


`$head->set_t_lemma($mwe)`
is the recommended shortcut for `$head->set_attr('t_lemma', $mwe)`


will try it thanks


Yeah, so apparently this wasn't doing what I thought it was doing. I thought this was an import+export statement where you were picking specific exports from cardContainer to export to the module, but really you picked these imports because they're needed to register the directive and controller.


This is why: http://stackoverflow.com/questions/18308700/chrome-how-to-solve-maximum-call-stack-size-exceeded-errors-on-math-max-apply


Yup, it can be.  And what you're suggesting should run faster, since the backing store of an STArray is now a `std::vector`.


Perfect, thanks.


这个指令是proxy发给discover的 所以应该没有必要case insensitive


I'd argue that the "1000" should be changed to a default parameter.


Call it `functionType`


Call it `functionType`.


pageload_metric will always test True in an if pageload_metric: test, so in run_job above the if measurement: will always be true.


@rogerioms ops! esqueci. Mas agora atualizei


It's usually fine to import from the astropy modules (resulting shorter lines and better readability), and usually only importing the classes we use in a test. So adding ``Table`` here would be sufficient.


@shakuzen Thanks for pointing that out. I have changed `@since` to 1.1. See a48864bf207fd2b0d98d33a2fc01e5456ca6d8c7


raxxPACK Plugin XCV AutoDetEdit.raxxPACK

Modify so that it will Play sound only if output is headphones


same


This breaks lists. For example, this list no longer renders correctly:

```
./hoedown  --special-attribute
* foo
bar

* foo
bar
```

It renders as

```
<ul>
<li><p>foo</p>

<p>bar</p></li>
<li><p>foo</p>

<p>bar</p></li>
</ul>
```

But it should be

```
<ul></li><p>foo
bar</p></li>
</li><p>foo
bar</p></li></ul>
```


I would suggest to use a method instead of function (not only here, but in general).
That means `my ($self, $head, @nodes) = @_;`
and call it as `$self->reconnect_descendants($head, @nodes)`
even if you don't need `$self` here now.
If you later decide to add some parameters to this block,
it will be easy to use `$self->my_parameter`.


@Freetown5 is it okay if I format Leadership pages to transition down instead of forward?


do a pull request into github pages


We'd still have a different one in the tree versus the one in the docs (which would need to generate it before the docs themselves).

Or, we can remove it from the tree and always generate before the docs (one extra step on every single `make check`.

What do you think?

IMHO, I don't think there's that much value in the diagram for the broad audience. @adereis also suggested to remove it.


The change to Item.js on line 296 includes the extra colon in the $.toaster title-
> title: 'Error:


@dwaynebailey

> I find the _new and _added distinction slightly confusing. I guess that relates to the fetch and pull distinction also. It might be more clear to have it illustrated somehow. Don't know for sure.

Yep im not sure of the best names.

In order to understand the best analogy would be git. If i add a file in my repo it is "untracked" which is the equivalent of "new". If i "add" it its now part of the repo but the changes have not yet been committed.

Perhaps we should call it "untracked" rather than "new"


If only one layer can be active at a time, there's no need to iterate through them. You can just set the activeModel to not be active.


to make sure it shows up regardless of the log level. POLA.


same.


in what sense? i don't follow.


TeamCity Colore :: Main Build [Build 140](http://tc.sharpblade.net/viewLog.html?buildId=683&buildTypeId=colore_mainbuild) is now running


I think the naming is confusing me a bit.

PerformanceData has a framework name and contains a list of "suites".

PerformanceSuite has a name, value and contains a list of "subtests".

runtest() returns an instance of PerformanceSuite. Inside of
runtest(), the PerformanceSuite instance is called a pageload_metric.

Each "subtest" added to the PerformanceSuite has a name from the
"page" and value from the median of the actual measurements for
"page".

The PerformanceSuite's value is calculated as the geometric mean of
all of the subtest's values.

I see from
https://github.com/wlach/treeherder/commit/8074a1434592155a7ae09e2c51f2fca2b7fe7a29
that PerformanceData is modelled after the PerfHerder artifact that
is to be submitted to Treeherder.

PerformanceData corresponds to the PerfHerder artifact.
PerformanceSuite corresponds to the items in the PerfHerder artifact's suites.

I think I would like to make the PerfHerder connection more explicit and to
rename them

PerformanceData -> PerfHerderArtifact
PerformanceSuite -> PerfHeaderSuite
PerfTest.performance_data -> PerfTest.perfherder_artifact

I'm not totally married to the CamelCase. Perhaps Perfherder?

I think I would prefer to have analyze_logcat collect the information and
return it to the caller who can then create the PerfHerderSuite which can be
added to the PerfHerder artifact.

Let's hook up on Vidyo and talk about it a little bit.


TeamCity Colore :: Main Build [Build 140](http://tc.sharpblade.net/viewLog.html?buildId=683&buildTypeId=colore_mainbuild) outcome was **FAILURE**
Summary: Compilation error: Corale.Colore\Corale.Colore.csproj (new) Build time: 0:0:17


PullLogicFromPlantComponents (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


Not really, when you write :

	rule test
		value += abc
		foo = $value

You don't have information who will call this rule and with what scope.
So you need to wait until rule is used in build operation to figure out scope.
And it's even trickier then :

	rule test
		foo = $value
		value += abc
		bar = $value

	build *.obj: test *.cpp
		value = bcd

Honestly I don't think this functional is necessary.
I can't guess use case where it's needed right now.
Ops in nested variables in build operations should cover everything we need.


Very true. Will update.


JavaScript JavaScript JavaScript!


Profile.js also has a double colon on line 679
> title: 'Equip:'


Alright, I've read a little more about this, and yep, your solution is the way to go. The main purpose of this mixin is really the `list-style-type` rule, we will almost always want to set margin and padding on `ul` and `li` every time, so we should just leave them out. It's ok like this with it set to `0` in the mixin, because overriding that _will work_ -- it was the selectors that caused the CSS reorder.

TL;DR -- :+1:


As you'll see, this doesn't currently pass the automated tests. I've traced the deps through on a VM matching the CI system and make the list
```
tlmgr install adobemapping cjkpunct environ fandol lualibs luatexja ms
tlmgr install trimspaces ulem xecjk xkeyval zhmetrics zhnumber
tlmgr install --no-depends cjk
```
where the `--no-depends` is needed as otherwise some extra font stuff is added. I'm still tracing one issue with the LuaTeX test (font is reported as having no chars).

Pull requests are a new thing for me, but I believe the 'correct' approach is for you to alter at your end. Could you also make the file names all lower case: that's the standard team convention.


nice


I agree - I much prefer the previous way. The main reason being is that the you can create the desired effect  above at an application level. Checking for a hit makes this a lot less flexible (Reverting back would also give a little performance hit as there would be a lot less hitTests going on).


Actually, this IS the equation needed to calculate profit/loss. Just need one final step to print out a profit/loss


You've closed the body tag here in your html, but the body tag should wrap all the content you wish to be shown. The browser will actually correct this and display the appropriate DOM, but when you start applying css, you could end up seeing some strange unexpected behavior.


call this `getExactWindowFunction` to be coherent with `getExactFunction`


Muito obrigado!


TeamCity ReOpenLDAP :: devel-2.4 [Build 197](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=854&buildTypeId=ReOpenLDAP_Devel24&guest=1) outcome was **SUCCESS**
Summary: Tests passed Build time: 01:42:00


Oh, nevermind. I forgot that for VirtTests the params won't be listed... hmmm....


clor — явная опечатка :)


related to 42f0e1f3de3006b9514f30829fc6f20d1bad4639


1 year for the CA seems pretty short. If this is for dev / reference, prob not a big deal, but I wouldn't want to have to re-roll entire pki infra every year.


Typo. You're printing `k` when you should be printing `d`


@pedromorgan Watch out to keep indent as spaces only and not tabs. I've made the correction


My bad.
For chapter 1.


TeamCity Colore :: Main Build [Build 141](http://tc.sharpblade.net/viewLog.html?buildId=684&buildTypeId=colore_mainbuild) is now running


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

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


A heading tag like h2 or h3 might fit better as this is a title of a new section of the content. Is there another area where you could use a heading tag to distinguish the beginning of a new section of content?


@anurse - I see now this on Travis on Mac:

Installed:
    1 package(s) to /private/var/folders/gw/_2jq29095y7b__wtby9dg_5h0000gn/T/pzuc6e8v.3a3/./bin/packages
    Microsoft.Dnx.Tooling.FunctionalTests.DnuRuntimeRestoreTests.DnuRestore_GeneratesDefaultRuntimeTargets(flavor: "mono", os: null, architecture: null) [FAIL]
      Assert.Equal() Failure
                ↓ (pos 0)
      Expected: runtimes/osx.10.9-x64/lib/dnx451/RuntimeR···
      Actual:   lib/dnx451/RuntimeRestoreTest.dll
                ↑ (pos 0)


TeamCity Colore :: Main Build [Build 141](http://tc.sharpblade.net/viewLog.html?buildId=684&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:0:58


You can force that by expiring the PHP session cookie in your browser. Does it work?


TeamCity Colore :: Main Build [Build 142](http://tc.sharpblade.net/viewLog.html?buildId=685&buildTypeId=colore_mainbuild) is now running


Something similar should be added to the conf/minion since these parameters are also configurable in the minion.


TeamCity Colore :: Main Build [Build 142](http://tc.sharpblade.net/viewLog.html?buildId=685&buildTypeId=colore_mainbuild) outcome was **FAILURE**
Summary: Compilation error: Corale.Colore\Corale.Colore.csproj Build time: 0:0:15


How about:
```
rule test
    foo = $value
    value = $value abc # translated from +=
    bar = $value
```
And no way to get `-=`.


@DefinitlyEvil it already is, or else Redstonelamp wouldn't work.

On Thursday, October 1, 2015, The Evil Man <notifications@github.com> wrote:

> @jython234 <https://github.com/jython234> And please put this new version
> on Maven. :D
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BlockServerProject/JRakLib/commit/30ebe4b8284da2d4a22a825c168124020e02f3c1#commitcomment-13542343>
> .
>

--


ahhh.. OK.. That was a kinda test commit of a virtualmachine+windows7..
And trying to figure out how to compile.. distribute etc...

So was kinda nervous which is why this is a test commit...

Need to set config in pycharm correct.. ..!!


Likely need this same change in ``AsyncEventPublisher``.


Que haces aca?


Thanks for adding this one!


TeamCity Colore :: Main Build [Build 143](http://tc.sharpblade.net/viewLog.html?buildId=686&buildTypeId=colore_mainbuild) is now running


I delayed the creation of the empty geometry until there was a problem. This avoids the new/delete happening with every feature that goes through the method.


@davekisly As you might have noticed, I finally merged my feature branch into master, any qualms about this particular commit or its parents? If not I will pull to prod.


TeamCity Colore :: Main Build [Build 143](http://tc.sharpblade.net/viewLog.html?buildId=686&buildTypeId=colore_mainbuild) outcome was **FAILURE**
Summary: Compilation error: Corale.Colore\Corale.Colore.csproj (new) Build time: 0:0:17


It is best to be as specific as possible when catching these exceptions. That way you don't accidentally catch an exception that you didn't want.


We haven't touched key rotation or ongoing maintenance at all, so TBH i don't think we can suggest the TLS infra established here should be used in prod.


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.487.7d833d163d0e49c97166eed46c119f0019513a5b](http://build.palaso.org/viewLog.html?buildId=63984&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:22:08


Since we're catching the IllegalArgument exception that is a HootException we get the QString convenience methods for free. I also threw in the element information since this isn't likely to come up too often and that info will likely tell us what is going on.


btw in the latest changes I have the CA expiring in 52 wks, and the certs expiring in 51


UM. Yeah. You got me there on (2). No, don't change it...


Should be fixed on latest dev/release branch


TeamCity Colore :: Main Build [Build 144](http://tc.sharpblade.net/viewLog.html?buildId=687&buildTypeId=colore_mainbuild) is now running


I think you're right. Fixed.


No problem! I think nearly all are fixed.


hey @mli , just wondering if you had a chance to look at this yet?  would be nice to get your help finishing this off if it's worth the effort.  thanks!


TeamCity Colore :: Main Build [Build 144](http://tc.sharpblade.net/viewLog.html?buildId=687&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:1:1


TeamCity Colore :: Main Build [Build 145](http://tc.sharpblade.net/viewLog.html?buildId=688&buildTypeId=colore_mainbuild) is now running


Шли PR, когда будет что стабильное


Done.


TeamCity Colore :: Main Build [Build 145](http://tc.sharpblade.net/viewLog.html?buildId=688&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:1:0


Yes, I found those by chance... This caused much delays just to get the error message that we don't support cudnn v1 anymore. Now it will compile and give the good error message with this fix.


Big thumbs up! :+1:


Miss follow up question.


Is this something you can own? Or should I create a pull request for it?


Should return the shorter one.


I'll run the time profile again. Do you still have the slow_compile.pkl file for this? I don't find it anymore. Otherwise I'll ask @t13m


ok so this is the same stuff to unpack on server..

The `ziped.ts2`  filename is a zip and will contain the large `simulation.json` file..

One think that occurs to me, is that we maybe need an `info.json` or alike.. with the "top level description"..
eg
```
sim_name: "Liverpool St"
description: "Lodon Liverpool street is a terminus blah blah blah
authors: [..] etc
```

This would be for the case where there ar lots and lots and lots of sims, and all we want to do is get the "meta data".. from a small fragment, instead of loading and decoding a huge blod of data"..
and similar to with HTML HEAD request..where we can reply with what it is, data, size instad of whole lot..


This ugly hack got me to display the virt-test params in the dry run log:

    --- a/avocado/core/test.py
    +++ b/avocado/core/test.py
    @@ -693,6 +693,7 @@ class TimeOutSkipTest(Test):
         def test(self):
             raise NotImplementedError("This should never be executed!")

    +
     class DryRunTest(TimeOutSkipTest):

         """
    @@ -700,3 +701,14 @@ class DryRunTest(TimeOutSkipTest):
         """

         _skip_reason = "Test skipped due to --dry-run"
    +
    +    def __init__(self, methodName='test', name=None, params=None,
    +                 base_logdir=None, tag=None, job=None, runner_queue=None):
    +        super(DryRunTest, self).__init__(methodName, name, params, base_logdir,
    +                                         tag, job, runner_queue)
    +        if getattr(params, 'keys', None) is not None:
    +            self.log.debug("Test parameters:")
    +            keys = getattr(params, 'keys')
    +            keys.sort()
    +            for key in keys:
    +                self.log.debug("    %s = %s", key, params.get(key))

Maybe there's a way to make test providers to provide customized dry run classes?


Very nice.
Results are clear and understandable.


@valib :+1:

 @SubJunk ums layout is designed for English. It's not great for others, especially for rtl, so saving size helps in this regard.  Have been looking into this in #657, will push stuff there soon I hope.


@pkasperczyk nie zapomniałeś dodać tego pliku do repozytorium?


Fixed in https://github.com/beaufortfrancois/sandbox/commit/4538f0eb20a388dee8b6caf9c707852f18ac5a30


static import


It is not changed to a default parameter, the value is just set in only one place, instead of 5 places.


I think you are missing the `'`s


This commit has an error in line 211 -- a sum function with null inputs should return null, not 0.0, as incorrectly assumed.  It is fixed in commit e5d1cdf259fcd6ec179e7d9cf44f96028eeb50d6 .

Read SPARK-5680 and SPARK-8828 to see the assumptions about how the sum() function should work with null inputs, how the bug was introduced in Spark1.4, then fixed in Spark1.5.


PullLogicFromPlantComponents (Myoldmopar) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Good, but not the best method.


Done.


Looks like there might be an issue with something on Linux and the Fandol font (perhaps a `luaotfload` bug). For the moment, could you add a conditional to bail out on LuaTeX until I can track that down: will lead to issues between platforms otherwise.


Miss follow up question.


@Nadahar, @skeptical thanks for support :-)
@SubJunk the problem with the certain size is first made by the translation and second by the windows font resizing depending on the screen resolution. It is obvious that some used English wording is shorter than the translation e.g. to the czech language which may makes the UI completely different than was intended. Also high screen DPI makes problems as Nadahar mentioned. See my comments at https://github.com/UniversalMediaServer/UniversalMediaServer/pull/616#issuecomment-137855541


I feel like this should be moved down.. it feels like it's the super important part of the lab being up here


Isso é pra conseguir extrair relatório de erros e utilização? Algo do tipo?


Removing this chunk doesn't look right.


Use StringBuilder instead of String in loop.


Thank you for your feedback.
I'll improve on those things.
Will we be able to know our score on each assignment?
Thanks


Good point. Just not really sure where to put it in. I guess I can slip it in right before the `description` override on `FISCard`?


Added a test. This is correct behavior: access to explorations within a collection is not inherited by the rights of the collection itself. This is why collections most likely will only be comprised of public explorations for now.


it would be nice to link to actual issues - for whatever reason github does not auto-link it.


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.5.65.7d833d163d0e49c97166eed46c119f0019513a5b](http://build.palaso.org/viewLog.html?buildId=63983&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 874, ignored: 50 Build time: 00:35:10


Could I update my solutions after the ddl?
> 在 2015年10月1日，下午5:09，Fei Yiran <notifications@github.com> 写道：
>
> Miss follow up question.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/flyoceans/CCAssignment/commit/603b351fce2451a937a2eb4a87c872e3335a02f1#commitcomment-13551059>.
>


Ah, you're right. I moved it to the top of the function to make that clearer.


lets bump to MIT here.


Ок, в принципе с попапом проблем нет, осталось реализовать `api.dom.confirmation` и возможно `api.dom.input`.
Про возможности, плз, проведи ревью и оставь критику, может подскажешь как это код симпатичнее порефакторить можно, или вообще другим способом все делать. Код довольно громоздкий получается, особенно из-за стилей в js.


ok, I think that accepting the defaults for this is fine at this point - from the docs, the file is stored in memory anyway - "until the file’s fileno() method is called" - which seems to be large enough (no disk write for a 250MB temp file on my system).


Lets use [Guava] (http://docs.guava-libraries.googlecode.com/git/javadoc/com/google/common/io/BaseEncoding.html#base64()) instead.

In general I would like to reduce dependency on the api-client  as much as possible.


It's always preferable to use `setTimeout` instead of `setInterval`.


added new readme


Uh oh. I do not think that this is going to fly... The problem is that you now expect a `SYSROOT` to be set, but we cannot do that. `git.exe` is a perfectly valid entry point for, say, PowerShell, where we can *not* expect any environment variable...


Ok, thanks for the fixes :)


It should work, as I added the magic literal handling to `getScalarFunctionImplementation`


Her mangler det fallback. Og en test på manglende felt 'o' (hylleplassering) som ikke er obligatorisk


Fixing it now ;)


What's the reasoning behind this change? I can see making the value configurable, but is there a particular reason to make the header name configurable?


Have better solution without using O(n) extra space.


LGTM


See https://github.com/beaufortfrancois/sandbox/commit/b86c8754b5f64a424948574fad1dfc6ee1412e62


if you have the time, a PR would be most welcome :+1:


Consider calling this from `componentDidMount`.


This code is a simple Banking Application written for a networking class.  There are UDP and TCP servers / clients, but both serve the same general purpose of sending messages back and forth between client and server.  If you want a better understanding of networking, I recommend Computer Networking: A Top-Down Approach by K&R.


I have your score now, and I will publish it ASAP.


Conditionals in JS where the test is equivalent to the if branch can be shortened. For example:

    entry.fields.traceNumber.value = entry.fields.traceNumber.value || self.header.immediateDestination.value.slice(0,8) + utils.pad(addendaCount, 7, false, '0');

Regardless, off the top of my head I haven't worked on this for a while so my recollection of why this might not work is non-existent. As far as I can tell this is fine. I'll have another more active developer than myself look over this.


We should make sure to unbind these events & kill any timeouts when this component is destroyed.


Miss follow up question.


Can you please make the style consistent here? Extra spaces present.


The marked check is done above, but the logging isn't added back there. Is it present somewhere else?


I think you misunderstanding the problem.
Check it again.


Er hylleplassering ikke-obligatorisk?


How is this different from https://github.com/git-for-windows/build-extra/commit/ec06b09b1cc3632e6419933ea198c7ce03d77b05, why did you not test my change as I asked, and how come I am all of a sudden signing off on a patch I never saw before?


Edicion del 01-10-2015


I would put this in the main conf, not the location conf. The reason behind this is that there is only one whitelist, so the cache size should be relevant to the one whitelist. Having it configurable per location seems like it would only lead to confusion.


Same as comment above. I think this belongs in the main conf


Perfect!


Fixed in https://github.com/beaufortfrancois/sandbox/commit/07827ead9689de48f7424cc90889cc1f8d8bd39d


virtualgl and manjaro-tools-iso added..


ランキング期間中に誰かによって提出されたErlangのコードは1秒台で文字数も少ないな
すげえわｗ


This commit references a memory leak fix, but it's not obvious to me where that is. Is it just due to the underlying librepsheet changes or did I miss something?