@alex based on your previous comments, we could take out the part of this that treats content-length=0 as only a query and simply check and handle when there is no body declared.

I could also refactor this to throw the original 400 error if they declare a body but there isn't one (this seems like a scenario where something is 'wrong'), but process normally if they provide a parsable body, regardless of the length stated (this seems like they just miscalculated or don't know how to set a content-length header and probably want the body parsed).

Open to thoughts


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


Yeah...
Unfortunately you do here the same: `release()` unconditionally.

Let me play on my side with the `tryAcquire` outside of the `lock` and I'll come back to you soon!


ts2 is not gonna change right ??

so maybe title needs to be,, in psuedo
```
APP_SHORT + " - "  + tr(APP_LONG) + " "+ VERSION
```


This branch is for cases like `SUM(DISTINCT a + b)`. At the last aggregate operator, `a+b` has already been evaluated since they is used as one of the grouping expressions in the previous aggregate operators.


That's a good question. It looks like the `usesTungstenAggregate == false` branch didn't contain this logic. Why wasn't it present there?


Contents of mcsv files :

* `vtkMRMLDoubleArrayNodeTest1_WithLabels.mcsv`

```
# measurement file /path/to/Slicer/Libs/MRML/Core/Testing/TestData/vtkMRMLDoubleArrayNodeTest1_WithLabels.mcsv
x,y,yerr
12.1,425.577,-454
8.79633e+09,0,-1
```

* `vtkMRMLDoubleArrayNodeTest1_WithoutLabels.mcsv`

```
# measurement file /path/to/Slicer/Libs/MRML/Core/Testing/TestData/vtkMRMLDoubleArrayNodeTest1_WithoutLabels.mcsv
# nolabels
12.1,425.577,-454
8.79633e+09,0,-1
```

* `vtkMRMLDoubleArrayNodeTest1_OldFile.mcsv`

```
# measurement file /path/to/Slicer/Libs/MRML/Core/Testing/TestData/vtkMRMLDoubleArrayNodeTest1_OldFile.mcsv
# columns = x,y,yerr
x,y,yerr
12.1,425.577,-454
8.79633e+09,0,-1
```


I need to select all posts (and all files) because output_page() counts how many "files are not shown" when printing the omit-message on the board page. It might of course be faster to select only the TOP REPLIES_PER_THREAD per thread in descending order. But then still the total number files per thread would have to be counted with an additional query.

Since the database seems not to be the bottleneck at the moment, I will definitely work on getting rid of the DateTime module next. Thanks for you comment. This is much appreciated.


update_unit_testing (mbadams5) - i386-Windows-7-VisualStudio-12: OK (1914 of 1914 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)


Nicely done.

Minor improvement: ```.items()``` returns a copy of the dictionary, ```.iteritems()``` avoids the copy.


saw approval from jvoegele
at https://github.com/basho/yokozuna/commit/4d8a71c7a7b8269f3096a8f593d5bbffa8262682


merging basho/yokozuna/merge/2.0-into-2.1 = 4d8a71c7 into borshop-integration-569-merge/2.0-into-2.1


basho/yokozuna/merge/2.0-into-2.1 = 4d8a71c7 merged ok, testing candidate = 2fbae049


You are probably confused with `dispatch_release`'s being all over the file. It is because I didn't bump `MACOSX_DEPLOYMENT_TARGET`when moving xctool to ARC and as a result ARC is disabled for GDC types. I will fix it later.


Last Updated


@danielmarbach I had to add this back in. This does raise the question if we need to add it as a header as well since users might assume that the messageid header is always there?

Another option would be to make the message id explicit on the incoming pipe and not rely on magical headers?


tooltip-append-to-body="true"


ng-if="explorationTags.tagsInTootlip.length > 0" (double quotes)


@jorgenavas , so there is no need to indicate when functions are entered and exited?


nit: add space after 'if'


nit: alphabetize


ah nvm. we do not need this branch with your change.


@levity something like this?


Turns out that we actually rely on it our selfves :(

https://github.com/Particular/NServiceBus/blob/develop/src/NServiceBus.Core/Pipeline/Incoming/InvokeHandlerContext.cs#L24


Şu commitler ne kadar düzensiz yazdığımı gsteriyor resmen :smile:


delete*


Okay.

You might still want to name these methods correctly with `Create` or `Copy` to follow convention and mark them with `__attribute((ns_returns_retained))`. Otherwise ARC may not know that you are returning retained objects.


Nice one @aidancasey :roller_coaster:


Care to submit a PR?


Could you update the dokany version used in AppVeyor builds to 0.8.0-RC2?
Presently the Dokan.NET builds on 0.8.0-RC1 still run into the 40min forced timeout.


This broke the travis-dmake test which relied on "make dmake" executing "./dmake" as well.
I already started wondering why it seemed so silence around this...


Yes it releases unconditionally to prevent any potential unknown bugs like the delayhandler where remove is called twice on the same message.
You had already moved tryAquire outside the lock so we shouldnt have any deadlock with both of the changes


It's common convention to have output parameters like `convertedSize` to be first or last, not buried in the middle.


how i do dis


This **must** be a `Map<String, Object>` as the values may not be strings at _insertion_ time, for example, they may come from a `Task<?>` via an `attributeWhenReady()` call, or similar. They will be coerced to strings at retrieval, maybe that could be more explicit. Suggest you delete this commit.


HOW U DO DIS


My bad, I was adding more and more arguments while working on the commits. Will fix.


thanks!


TODO: Remove


It looks like you replaced the ucevent symlinks with real files?


sell?....wrong verb, send is more appropriated hahaha


See commit 69af9c


This looks really nice, really like it!


:beers:


Now I remember, the ZerosReader is to make the test suite pass, see:
https://github.com/aibaars/distribution/blob/0d482a366e8d1927806775916e5391490b01eab4/registry/storage/driver/testsuites/testsuites.go#L416

I'd much rather return an InvalidOffsetError instead of writing a broken file, but I also like to pass the test suite.

As far as I understand the test cases seem inconsistent with the documentation of the storage driver: https://github.com/aibaars/distribution/blob/0d482a366e8d1927806775916e5391490b01eab4/registry/storage/driver/storagedriver.go#L61 which states that:
	// The offset must be no larger than the CurrentSize for this path.


Done


nice, thanks for edits.  seconded on maybe a new video illustrating the react-router updated syntax?

awesome work, btw.


Removed usage of JSON API (replaced with call to StatObject)


We will need to add these changes before doing the pull request on the private KualiCo Financials repository. These packages do exist in that repository.


So for now I'll just leave the ZerosReader.


What will it require to remove the usage of the modulestore in the `transform` phase?  All modulestore-dependent code should reside only in the collect phase.  This will allow us to eventually separate LMS (cached-read-only-courses) and Studio (read-write-courses).


i will push this to master, and remove references to allowed/online cpusets.

as a side note
when I have some more time, I will do the porting so ompi can use hwloc master.
once references to offline cupsets have been removed, I ll have to add some #ifdef to handle cache objects
(future hwloc is really straightforward from an ompi point of view)


still bootloop:P


Do you have a fever? ;)


A few rough comments:

1. One should think carefully about combining monad transformers with free algebras (e.g. `M` above). There are definitely use cases, but sometimes it's semantically more correct and easier / faster to extend the algebra (e.g. `PathErrorF`) instead.
2. Technically you can define the dsl methods in terms of any other algebra which is "broader" than the required one (via an implicit `Inject` parameter). This way you get back `Free[F, ?]` for some broader algebra `F`, which can make it easier to compose (note: this comes from my experience with PureScript, not Scala, so YMMV).
3. Death to OOP! :smiling_imp:


Woohoo! :+1:


Seriously, the PR diff now becomes readable. It was a long shot, but worked out ok :-)


With input data being sanitized, `$data['make2x']` evaluates to boolean `true` rather than a string.


@lpshane, you are my here! +1

Let's hope Cordova picks up on this one and provides an official and more generic solution soon!


lg


did a quick try to see how the flow works all in hand drawn style. also many of the reference stuff feels like it can be better positioned later in the piece. it's not specific to the introduction.  check out this issue to see why i made some drastic changes.  https://github.com/tchoi8/handmadecomputer/issues/6


all tests pass:
success: http://buildbot.bos1/builders/test-yokozuna/builds/1379


I think what you have is good.


I'd like to congratulate @hugobessaa, i've  been mistaken with him in several opensource projects already. That means he's very active hahahahaha.


This import is causing cyclic import errors for me now. I think, in general in the [MVC] (https://en.wikipedia.org/wiki/Model%E2%80%93view%E2%80%93controller) pattern, you don't want the model to know anything about the viewer.


I like not to hardcode urls and use their names so i can mess with routing without breaking tests.
Plus, you never know when your api is gonna end up in the subdirectory of a dark cave.


Oops. Didn't notice that. Sorry.


correction: there are separate *nodes* for image and text. The image area is a Pane, but the textArea is a Node (Pane and TextArea are subclasses of Node)


Commit message still uses 'selector' node.


That will probably skip PRs altogether?!


This fixed #2


No, travis builds PRs, this forces travis to not build any branch which is not a PR ;)


@k00ka Please open an issue and provide a script we can use to reproduce the issue using [this template](https://github.com/rails/rails/blob/master/guides/bug_report_templates/active_record_master.rb), as per our bug report guide found [here](http://guides.rubyonrails.org/contributing_to_ruby_on_rails.html#reporting-an-issue)

Commenting on an almost 2 year old commit is not the proper way to report an issue or get attention on a bug.


update_unit_testing (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (840 of 840 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.3%25-green.svg)


This function name change is wrong, please restore the original `getFroalaEditableJQueryInstanceMethod` function name here.


Will this work for opensolaris? osx can have libresolv (I'm not sure). QNX, for example, requires libsocket: http://www.qnx.com/developers/docs/6.3.2/neutrino/lib_ref/r/res_query.html

I'd prefer more common solution. Maybe selection of library (or absence) depending on OS; and at the end try to build simple source code with `#include <resolv.h>` and `res_query()`. If you would like to work on this, you can see how to build source code here: https://github.com/boothj5/profanity/blob/master/configure.ac. I not, I can do this later.


That would be sweet then.

I've contacted support a while ago for the same reason (to basically skip builds for the same commit / head, and there was no solution for it at that time, although that option probably existed already).


e


хахаха путед :D


Are then only PRs against master being built?  (which would be fine, just out of curiosity).


done. thanks :)


Even without when functions are entered and exited cpa witness checker can
do its job for very tiny programs. But I need to check with svcomp
programs. Even if it doesn't complain it might get lost without that
information.

On Wed, Oct 7, 2015 at 2:51 PM, Arie Gurfinkel <notifications@github.com>
wrote:

> @jorgenavas <https://github.com/jorgenavas> , so there is no need to
> indicate when functions are entered and exited?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/seahorn/seahorn/commit/04b2648c89ea9c6f1851a701c6cf2e559dd1872b#commitcomment-13650349>
> .
>


Haven't reviewed the code yet, but would suggest adding those comments to the commit message with git commit --amend.


Maybe this should be squashed on commit that adds string generator.


Hey, give me your email and I'll put you in as a contributor.


@jfoug with this commit, plugin_deps.pl got recursive. It keeps its own "uniq" list so will not go in any infinite loop.


Shouldn't you just move this to line 13?
```php
if (!$info['topicId'])
    $info['topicId'] = isset($_GET['ht']) ? $_GET['ht'] : $cfg->getDefaultTopicId();
```


Shouldn't this be :redis => server instead of :redis => Redis.new()?


Should we have a different log message here?


This fixed victorhu33/ProyectoLosCremoleros#181


As I said, it might be just re-verifying. I don't know how to tell whether
it is actually using the witness or not.

On Wed, Oct 7, 2015 at 6:10 PM, Jorge Navas <notifications@github.com>
wrote:

> Even without when functions are entered and exited cpa witness checker can
> do its job for very tiny programs. But I need to check with svcomp
> programs. Even if it doesn't complain it might get lost without that
> information.
>
> On Wed, Oct 7, 2015 at 2:51 PM, Arie Gurfinkel <notifications@github.com>
> wrote:
>
> > @jorgenavas <https://github.com/jorgenavas> , so there is no need to
> > indicate when functions are entered and exited?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/seahorn/seahorn/commit/04b2648c89ea9c6f1851a701c6cf2e559dd1872b#commitcomment-13650349
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/seahorn/seahorn/commit/04b2648c89ea9c6f1851a701c6cf2e559dd1872b#commitcomment-13650736>
> .
>


I think these should be equals not +=


one minor comment


Shouldn't this be Redis.connect(...) instead of Redis::Namespace.new(...)?


Just googled it to make sure, Looks like its a different spelling for US and UK (I'm in the UK)


let's take out all the puts'es on the next commit


=)

On Wed, Oct 7, 2015 at 5:37 PM, Kanstantsin Shautsou <
notifications@github.com> wrote:

> Yeah, got it. Simple docker_image pull was broken.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bflad/chef-docker/commit/24b40f0df7b2092d71b0e1ef1508fdf75303e149#commitcomment-13650066>
> .
>


Bwah ha ha. Fine copypasta.


hmm. interesting. i will try this, and see how it's goes.
Thanks.


i think you are right, though I doubt this is the cause of deadwing's issues (LowerEnmityByPercent isn't used in many places)


Since `service/coerced-trap` was wrapping the response in `success-response` or `ok`, I think these `coerce!` calls still need to be wrapped in an `ok`.


:+1:  Awesome! Thanks!


I think it would still test any PR.


This is a test message on that line.


function view(string $name, array $data=[], array $options=[])


Nice Comment ! keep it up.


Change looks good too.  :+1:


`staleHosts[uuid] = staleHosts[uuid] + 1`


This is all fixed up - thanks @jsg2021


The package changes may need to be added in the submit branch that has the latest KualiCo code.


There is no replacement for this section?


when I run timedelta like this, I get
>>> t = time.time()
>>> t2 = time.time()
>>> datetime.timedelta(t, t2)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
OverflowError: days=1444272671; must have magnitude <= 999999999


TODO: add this note to the copyright section within the text.


can probably move delete to bottom


The saved names aren't locale dependent - its a config name, so its static across all platforms and languages.


This change is overwriting my changes because you have old copy of this file.


It something wrong on Windows with this line.
I need to change it to + packageName.replaceAll("[\\./]", File.separator) because codegen failed during genertion process for config.json with following config.json:
{
 "modelPackage": "App\\apiModels\\travel\\v1\\prototypes",
 "srcBasePath" : "",
 "invokerPackage":"App\\apiModels\\travel\\v1",
 "packagePath": "App\\apiModels\\travel\\v1",
 "packageName": "App\\apiModels\\travel\\v1",
 "apiPackage" : "api"
 }
It finish with error:
Caused by: java.lang.StringIndexOutOfBoundsException: String index out of range: 1
        at java.lang.String.charAt(Unknown Source)
        at java.util.regex.Matcher.appendReplacement(Unknown Source)
        at java.util.regex.Matcher.replaceAll(Unknown Source)
        at java.lang.String.replaceAll(Unknown Source)
        at io.swagger.codegen.languages.PhpClientCodegen.toPackagePath(PhpClientCodegen.java:122)
        at io.swagger.codegen.languages.PhpClientCodegen.modelFileFolder(PhpClientCodegen.java:216)
        at io.swagger.codegen.DefaultGenerator.generate(DefaultGenerator.java:155)

Problem is with parameter:
"modelPackage": "App\\apiModels\\prototypes",

When I correct this line to:
+ packageName.replaceAll("[\\./]", File.separator)
codegen works correctly


Actually my bad! Misread the "!=" part of the clause as "==". Makes sense now. LGTM!


Hey man thanks for all corrections

We can't be sure you're right but if don't get started in that time
interval downtime command will gracefully fail, though it was tested in
various circumstances with different VMs and we did't have a single failure
I think we don't need that sleep line anyway but maybe we can do something
like this:
```bash
if [ ${OCF_RESKEY_migration_downtime} -ne 0 ]; then
    while [ -z $lm_pid ]; do
        sleep 1
    done
    ocf_log info "$DOMAIN_NAME: Setting live migration downtime for
$DOMAIN_NAME (using: virsh ${VIRSH_OPTIONS} migrate-setmaxdowntime
$DOMAIN_NAME     ${OCF_RESKEY_migration_downtime})."
    virsh ${VIRSH_OPTIONS} migrate-setmaxdowntime $DOMAIN_NAME
${OCF_RESKEY_migration_downtime}
fi
   ```

OR something similar but with some time interval

```bash
if [ ${OCF_RESKEY_migration_downtime} -ne 0 ]; then
    lm_counter=0
    while [ -z $lm_pid ] && [ $lm_counter -lt 10 ]; do
        sleep 1
        let lm_counter++
    done
    ocf_log info "$DOMAIN_NAME: Setting live migration downtime for
$DOMAIN_NAME (using: virsh ${VIRSH_OPTIONS} migrate-setmaxdowntime
$DOMAIN_NAME     ${OCF_RESKEY_migration_downtime})."
    virsh ${VIRSH_OPTIONS} migrate-setmaxdowntime $DOMAIN_NAME
${OCF_RESKEY_migration_downtime}
fi
```

What do you think and do you have any proposal?

Thanks once again man

On Oct 7, 2015 4:28 PM, "Dejan Muhamedagic" <notifications@github.com>
wrote:

> On Wed, Oct 07, 2015 at 07:05:42AM -0700, psyhomb wrote:
> > This is just a precaution measure because downtime interval can only be
> set
> > while migration is running so I wanted to give some time for migration
> > process to start and make sure it will catch it during its run
>
> OK, but sleeping for 2 seconds seems like an ad-hoc measure. How
> can we be sure that the migration is going to get started withing
> 2 seconds? Is there perhaps a better way?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/psyhomb/resource-agents/commit/9d7bc104869ba24d267d621de8cfa918c8f8a366#commitcomment-13639873>
> .
>


I'd like to keep the FileLength limit on 2000, and only ignore on a case by case basis.


Our situation may be unique, but we want all json responses to be json+hal, so we are registering the JsonHalMediaTypeFormatter as the first formatter.  Since it extends the JsonMediaTypeFormatter if you request application/json the json+hal representation is returned, but the content-type is application/json which is incorrect.  If you register the JsonHalMediaTypeFormatter at the end of the formatters list, then someone requesting application/json would receive application/json from the JsonMediaTypeFormatter and the content-type would correctly be application/json.


ctimer_init does a couple of other things before starting the process, so I think we should still invoke it.


not that i want to stifle initiative, but why would you do that last side note? we can make the adjustments when we do the update. otherwise, we'll just have to go thru and remove #ifdef's all over the place, which doesn't sound very attractive.


this changes only the M3 parameters (an auxiliary data in standard reco).
On the other hand in HLT there is a HLTrigger/Configuration/python/customizeHLTforMC.py which overrides these parameters to something else.

Should the customizeHLTforMC be removed or not?


Do we have the same bug in another JavaScript repo and Java ps client repo?


namespaceCode+componentCode+parameterName to retrieve your parameter is overkill. It is customary to use either namespaceCode+parameterName OR componentCode+parameterName to retrieve your parameter. Please use namespaceCode+parameterName to retrieve your parameter as that will make the change to class DistributionOfIncomeAndExpenseElectronicPaymentClaimingHelperStrategyImpl.java unnecessary.


I think the part in parentheses stating "which is the 'handle button click' response" needs to be updated; that comment was specific to continue interactions.


Unneeded. See notes on parameter retrieval below.


@SimonCropp What made these test dodgy in the first place? It looks like they are testing that stuff gets moved to the appropriate place when it fails. I'll admit that doesn't really seem to be testing the Unity stuff. Is this an exercise in reducing superfluous testing of the code base?

If so then mission accomplished go ahead and merge it.


inb4 "use arrays" to point out that POSIX doesn't have arrays.


@paulmey could you please do a quick review ?


consider using a method or loop


Hi @posita

> Your code happens to pass the tests, not because it honors the spirit of what is needed, but because it allocates and returns new "null" objects directly from the handles after they've been closed. Unfortunately, the expensive resources are still hanging around. So now we have a leak and our current tests are not well-written enough to detect that.

Sure :)

> Does that give you a better idea of what is expected? If so, can you write better tests to detect the flaws, and then get them to pass by making corrections to the code?

Sure. It's done! Please check the new building .. it is available at npm too.


Résolution des conflits dans TestRoom


> I'm not any of those three, but I checked and can confirm it is supposed to be percentages in TBC also.

Health, yes. But what about power? The millions of Lady Vashj videos taken in 2008 show an absolute value, not a percentage. And they perfectly match the data given out in the Bestiary.

https://youtu.be/TwiIOq8xri8?t=206

Here is a video of Buru taken in July 2006. You can see that the health bar is constantly changing due to calculations, the mana bar remains fixed at 471600. (Bestiary -> 471600)

Here is another one, with the default ui.
https://youtu.be/rvWB4V7g-Pc?t=536


Według moicg obliczeń brakuje pozostałych dwóch osób #CzekamyCzekamy


so we can build ompi with --with-hwloc=,,, with both v1 and v2
otherwise, once we move to hwloc v2, we can no more build ompi with hwloc v1
makes sense ?


@AngieHinrichs This is something new for me, thanks for the tip, FAQ updated.


Thanks!


Nit: Parentheses around the conditional part of the conditional operator.


you want datetime.timedelta(seconds=self.end-self.start)


the rest of the clients are good


This is never true, see the `while` expression.


Interesting, I'll add a comment about that. I just made the log private, but I'm also unhappy. I'll revert the relevant part.
BTW, the API key is also logged anyway served side, since it's a URL parameter. I've created ps-tuebingen/handin-config#7 for this.


move to top of the loop


ah, i see - yes, that would be required. Thanks!


Grade for days 6 and 7: 16/16 - Nice work!


Are you just worried about being inefficient or something?


I'm a bit confused by the semantics of this property. How does an answer_type of 'Null' indicate that only one possible answer may be submitted for the interaction?


Probably want to drop this `puts`. :+1:


> One should think carefully about combining monad transformers with free algebras

Yeah, this is definitely an area that isn't obvious to me on which way is best to proceed. My current thinking is to include the error as a value (and thus possibly use a monad transformer like `EitherT`) when the failure is part of the domain. i.e. when opening a file for reading, it seems to make sense from the semantics of the file system abstraction that this could fail when the file does not exist, thus we'd return `PathError \/ FileHandle` in the algebra. Contrast this with say, a networking error causing a read to fail which, though still an error, is related to the particular implementation of the interpreter, not part of the domain itself. Does that make sense?

@jdegoes Any suggestions/comments on how one might handle this sort of situation via extending the algebra? Or would one just push all the errors down to the interpreters?

> Technically you can define the dsl methods in terms of any other algebra which is "broader" than the required one

Aye, we're already doing this: `final class WriteFiles[S[_]](implicit S0: Functor[S], S1: WriteFileF :<: S)`, the `WriteFileF :<: S` means `Inject[WriteFileF, S]`. Though maybe this means we shouldn't be using the infix type operator as it's meaning is less obvious =(


@delivery approve


@balloob Interesting, I've never seen the bit on acronyms. Guess I learned something new :)


@posita

Implemented a counter at `NAMESPACE` level to handle the number of active resources and, if there is no active resource, we delete the `expensiveresouce`... the test is passing ... but, I have sure there is some leaks in terms of when we have resources with different ids .. I will check it


Here would be the proper location to check whether the parent function is static and return NULL in that case if `stopOnStatic` is set.
That triggers an LLVM assertion (`Can't get a pointer to <null> type!`) in `DtoNestedVariable()` though.


da


Unless I am mistaken, I think this passes for the following situation:

is_linear: True
is_terminal: True
default_outcome_heading: None

Perhaps there should be an assert ensuring that linear interactions are not terminal.


eta devka slovno pulya


> Why is QGISDEBUG defined but Q_ASSERTs are not evaluated? Shouldn't QGISDEBUG be undefined in that scenario anyway?

I have no idea why the use of 'it' inside of the Q_ASSERT bool parameter isn't noticed by the compiler after expanding the macro. This is compiling with clang on Mac from inside Qt Creator 3.5.0, with `-DQGISDEBUG=1`, so the definition is there. Not sure how it would be undefined in this situation.

> Just wondering if fixing the problem at another point would have a bigger effect, avoid having to fix things like this again in the future and avoid the useless extra call to find().

Not sure I understand what fix in another place would keep this warning from showing up. I think it is localized to the Q_ASSERT macro and its expansion (or lack thereof). Should expand to the following (which should contain the variable in the condition) after preprocessing (from `/src/corelib/global/qglobal.h`):

```C++
#define Q_ASSERT(cond) ((!(cond)) ? qt_assert(#cond,__FILE__,__LINE__) : qt_noop())
```


The empty anchor element does seem a bit weird.

Incidentally, this commit is mixing concerns. Fixing the spelling violation and remove the suppression is one thing, the reformatting of the remarks element is another.


Do we have any designer files that were causing violations?


That works too :+1:


Does this change anything? Resizing mainContainer doesn't change anything about the text objects.


I created a PR for this: https://github.com/zurb/foundation-rails/pull/128


Is Box:resize super important or is it just for convenience?


Can you also add pinout for the USB? You should be able to use the same naming as on scarab-soc:
https://github.com/enjoy-digital/scarab-soc/blob/master/platforms/minispartan6.py#L64


Squash _is_ primarily.. ?


I think container:wrap should call container:relayout, since it's supposed to mean that it automatically shifts to "wrap" the children. Actually, it might not be a bad idea to just remove the adding children prat from container:wrap altogether, as that functionality is already covered by container:add.


*10000, not 1000


Why would you have that many undesirable room mods?


Does this do anything? Resizing mainContainer shouldn't change anything about the text objects.


iteration 4


NO.


partout dans le backend c'est seat_num


You may want to resize the object, then you should recalculate the layout (in case of container at least).


this seems to overrite my 2015-2016 season additons

<option value="2015">2015/2016 Season</option>


True. I wanted to separate object creation (everything up to this function) from layout definition (the body of this function). In the future the layout could be defined outside the program (like in xml/json file).


OK, ignore above comments. After reviewing the possible "getParameter" combinations, I think you are doing this the right way.


> I have an improved code, in case you are curious or ever need it

Integrated - thanks!

![image](https://cloud.githubusercontent.com/assets/223702/10353242/cbff99cc-6d4c-11e5-9247-624acdd33af2.png)


yeah, it's because you put it in my repository, but the gui messed up and
didn't download it so when I updated it everything got complicated. I have
to put the original files back in now.

On Wed, Oct 7, 2015 at 5:37 PM, Zigmas Maloni <notifications@github.com>
wrote:

> this seems to overrite my 2015-2016 season additons
>
> 2015/2016 Season
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NCDA/RPI/commit/468416a7f839ce312679a88d463a036202d4e222#commitcomment-13651210>
> .
>


OK, ignore above comments. This is needed.


Whoops, good catch! Fixed.


Done.


Currently, "Select your rosterData.txt to import." is translated as "选择需要导入的花名册.txt文件。".

Should I change "花名册.txt" (which apparently is "roster.txt") into "rosterData.txt" (written in English in the Chinese version) ?

The line would become
```
title.import.roster=\u9009\u62e9\u9700\u8981\u5bfc\u5165\u7684rosterData.txt\u6587\u4ef6\u3002
```
instead of
```
title.import.roster=\u9009\u62e9\u9700\u8981\u5bfc\u5165\u7684\u82b1\u540d\u518c.txt\u6587\u4ef6\u3002
```


i don't think it is less readable this way.


Looks good, thanks man.


oh, i wanted to grab the most recent files from the vonstuben branch to work on the displays


Can you define the MT41K128M16 in this target file for now? We will integrate it in misoc when validated.


Geil! Genau nach meinem Geschmack! Würde auch echt gerne teil deiner Idee werden und könnten auch Aspekte von meinem kombinieren :+1:


gonna update them now

On Wed, Oct 7, 2015 at 5:44 PM, Zigmas Maloni <notifications@github.com>
wrote:

> oh, i wanted to grab the most recent files from the vonstuben branch to
> work on the displays
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NCDA/RPI/commit/468416a7f839ce312679a88d463a036202d4e222#commitcomment-13651300>
> .
>


If the answer type is Null, this means that the answer is basically ignored (its only value is None), and that all we care about is that an answer is submitted at all.

Having said that, I think maybe we should invert this? Basically 'Continue' etc. should not have an answer type, and we should make is_linear a primary property of the interaction that defaults to False. When evaluating an answer, we first check whether the interaction is linear; if it is, we return the default response; otherwise, we look at its answer group and use that for the rule evaluation. What do you think?

Also, this would be a more involved change. What are your thoughts on doing it in a separate pass?


Stealth 翻成潛行，目前的我都改了，之後有遇到就用潛行
一般遊戲裡潛行和隱身是指不一樣的東西，效果也不一樣，
潛行有點像匿蹤，通常是盜賊或弓箭手擅長的，隱身通常是指法術造成的。


Du musst dich ja nicht auf eine Idee festlegen. Kannst ja mal sagen ob du eher 2D oder 3D machen willst.


Yup, that sounds perfect - I hadn't noticed that the file name was translated (even though it shouldn't have been).


I think it is better to first wrap (add) all children, then relayout when needed. Imagine you have added 5 children, then added another 5 children (before drawing for the first time) - the first (automatically called) relayout would be a waste of cpu cycles.
Another idea I had is to have drawboxes.createLayout(function() ... end), and then drawboxes:update() would relayout the whole layout only if a self.dirty flag was set. And that would be set by any resize()/set* function. Then wrapping() would only set dirty flag.


No, because each of our threads in the pool would still be using the same Redis connection.


Good catch, putting a `super()` here would actually cause some strange errors.


Ich denke ich habe dein Konzept verstanden. Aber wie meine Vorredner kann ich es mir doch nicht so ganz vorstellen.
Hoffe du zeigst uns noch was du meinst! :+1:


the new git header should be `# $Id$`


Looks like this commit includes a merge conflict...


python3_3 has been announced for deprecation, please don't add it to new ebuilds/versions


Ah, good call. I cleaned this up a bit.


isn't it incomplete should not you return the field xpadnl_max_named

````
size_t
PadnamelistMAXNAMED(padnl)
       B::PADNAMELIST  padnl
    CODE:
        RETVAL = padnl->xpadnl_max_named;
    OUTPUT:
       RETVAL
````


Have you forgotten to "git add" smf_addip.c?


pushd and popd can fail as well and should have `|| die`


Echt cool! Erinnert mich leicht an la noire, aber deine Idee klingt sogar besser. Wird man schön zum Denken angeregt.
Würde bei deiner Idee bei der Umsetzung mitmachen :+1:


Ha, thanks for catching these... definitely issues with trying to type late at night.


All set, go for it

On Wed, Oct 7, 2015 at 5:45 PM, Vanswa Garbutt <vgarbutt@gmail.com> wrote:

> gonna update them now
>
> On Wed, Oct 7, 2015 at 5:44 PM, Zigmas Maloni <notifications@github.com>
> wrote:
>
>> oh, i wanted to grab the most recent files from the vonstuben branch to
>> work on the displays
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/NCDA/RPI/commit/468416a7f839ce312679a88d463a036202d4e222#commitcomment-13651300>
>> .
>>
>
>


Fix provided in #1


update_unit_testing (mbadams5) - Win64-Windows-7-VisualStudio-12: Build Failed

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


We will maybe have to create a a7ddrphy and maybe do some adaptations since I'm not sure artix7 supports ODELAYE2. I'll probably do it if this is the case since it will be easier to do a modelsim simulation with the DDR3 model from micron.


LGTM


r+


Does vim-mkdir do more than make `:e new/directory/file.rb` work?


Incompetence


Echt cool. So ein Gruppenkalender kann man immer gebrauchen. Mit einem Wecker oder Erinnerungsdings wäre das super.


IIRC the Q_ASSERT macro is like a noop when QT_NO_DEBUG is set (i.e. release).

I guess there's no way to avoid to put Q_UNUSED after the Q_ASSERT if the variable is unused out of the assert condition.


saw approval from jedmccaleb
at https://github.com/MonsieurNicolas/stellar-core/commit/0574101a52dd7fd0e5d12bf50256ee1c202cea5c


merging MonsieurNicolas/stellar-core/SCPState = 0574101a into auto


MonsieurNicolas/stellar-core/SCPState = 0574101a merged ok, testing candidate = e78aebbd


this can safely be set in global scope, pkg_setup is a bit uncommon for this


Fußball Kartenspiel klingt super. Zwar nicht mein Gebiet, aber viel Spaß!


Fix works and branch can be merged into develop.


Ha-ha!
I've made it better (i think) yet again.


you're right. the pydoc documentation is rather sparse for timedelta. I'll update and send to the list


Top :+1:
Hatte auch erst so einen Gedanken!


Sucht einfach mal nach "Hack RUN". Dann habt ihr vielleicht ne bessere Vorstellung. :)


@bFekete should the cell and virtualhost name be hard coded here?


Comment should refer to IP model, not PCA model.


Ok, done new tests using different ids and proving that expensive resources are being cleaned up properly


pushd and popd can fail too and should have `|| die`


@sklum do you have anything that is releasable or should I tackle this when I have time?


Was just copying the same thing already existing in the ebuild. Feel free to change this to be better. I'm not the maintainer of it.


done


That's cool, good to put it in a partial because that is consistent with the twitter link.

I was about to nitpick about the name `object`, which could be more specific, but then I saw it is the same in the twitter partial, so at least it's consistent. If you feel like improving this, it would be cool if you could give that variable a more specific name (object is probably the most generic name possible. Why not e.g. `person`?). If you want to leave it like this for now, that's fine too.


and he even revbumped :+1:
:D


don't copy errors! ;)


Gar keine schlechte Idee! So haben kleine Gruppen die Möglichkeit sich zu etablieren und sich einen Namen machen. Hoffe du kannst diese Idee verwirklichen! Viel Spaß damit :+1:


How would you feel if we just `import { render, unmountComponentAtNode } from 'react-dom'` ?


@flavour {\"cat\"

This ones just escaping the double quotes, right?


@hlynurf allar þessar format breytingar hérna.. er einhver shortcut til að formata allt skjalið eða hver er fljótlegast leiðin til að gera það?


Verstehe ich es richtig? Ein Spiel, welches durch die Sounds gelenkt wird? Hört sich interessant an :-1:


adding blank lines


Yes, that sounds much cleaner and obvious. I like that suggestion.


This is basically equivalent to what the conversion would have generated, and I will move it there, but I got into a situation where the generator wouldn't generate the conversions because of the state the source tree was in.


af hverju ekki að eyða þessum kóða út? eða ertu mögulega fara nota hann seinna?


Hi @yaniaular,
Those issues that you commented above have been solved with https://github.com/vauxoo-dev/rma/commit/d3950fea9ecc968a97c12a413b06b2c6601fa252

Let me know your comments about it


This broke the toggles on my phone. Samsung Galaxy S6 running 5.1.1


sounds good


Should probably be resolved in d2 where on Model.create(); we initialise all (required) boolean values to `false`. https://github.com/dhis2/maintenance-app/issues/22


ifdef default behavior too?


Solche Spiele stehlen meiner Meinung nach den sozialen Aspekt solcher spiele ^^
Ich finde deine Idee keineswegs schlecht, nur anschließen würde ich mich nicht und wünsche dir viel Erfolg! :+1:


Yay!!!


Oops, thanks.


This looks wrong.  Shouldn't we be importing <unordered_map>?


Bom, pelo menos ficaram mais bonitos os testes ahaha


Nope.


Oh I see. Here's where it's done.  You can disregard my previous comment.


Fixed.


Holy crap, does this seriously actually work??

I was so sure the JNI bindings would brutally segfault or something...


`#include <unordered_map>` and then use just `std::unordered_map`, no `tr1` necessary.


eee ameeoo


Nit: Possible source of confusion. 'None' could imply the string value "None" like it was before, or the actual None python value. Since it's changing from 'None' to None, I think it might be worth removing the quotes here to be more clear.


This should be `this.detectSrvHop` instead of `this.stLoadBanConnect`, now it's overwriting the wrong tooltip and not displaying one for the server hop feature.


This is just a demo.  I think we are making headway.


Update README type variable names.


If so the check should be for QT_NO_DEBUG instead of QGSDEBUG?


Well, isn't it is your deal?

On Thu, Oct 8, 2015 at 1:49 AM, Ralph Samuel <notifications@github.com>
wrote:

> Looks like this commit includes a merge conflict...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/catflight/woopra-ios-sdk/commit/80fe0bf3de97df24e9337b1e2ca202cb536c1916#commitcomment-13651373>
> .
>


@wemrysi Well, if it's part of the domain, then I would extend the algebra with an error case, e.g. a `PathError` operation in the file system algebra. Anything encoded via return values will be opaque to interpreters, so it prevents applications such as logging of errors. If you force it into the algebra, then the interpreters have visibility and you gain new power there.

The tradeoff is that if you extend the algebra to support failure, then potentially *any* operation can fail, which may be less precise than the intended API. But in this case, it's already happening since all the methods return `M` so I don't see that tradeoff here.

> Aye, we're already doing this:

Ah, you're doing it "module fashion". Any reason to prefer that (e.g. better type inference) than "function style", i.e. simply having a collection of methods, each of which takes an `F` and `Inject`?


Here is a slightly re-organized message ... it could be tweaked further:

```
BUG: Fix support for missing labels in vtkMRMLDoubleArrayStorage reader

This commit updates the reader/writer of measurement file associated
with vtkMRMLDoubleArrayStorage writing the line '# nolabels' in the file
if there are no labels.

If the reader finds this line, it will correctly set the first line of values as
the first data point, instead of setting it as label values.

Remarks:

 * It changes the file format and ensures older file can still be loaded.

To help understand the motivation, here are four examples with:
* (u,v,w) as labels
* (12.1,425.577,-454) as  first data point,
* (8.79633e+09,0,-1) as second one:

(1) New writer: vtkMRMLDoubleArrayNode with labels

   # measurement file filename.mcsv
   u,v,w
   12.1,425.577,-454
   8.79633e+09,0,-1

(2) New writer: vtkMRMLDoubleArrayNode with no labels set

   # measurement file filename.mcsv
   # nolabels
   12.1,425.577,-454
   8.79633e+09,0,-1

(3) Old writer: vtkMRMLDoubleArrayNode with labels

   # measurement file filename.mcsv
   # columns = x,y,yerr
   u,v,w
   12.1,425.577,-454
   8.79633e+09,0,-1

(4) Old writer: vtkMRMLDoubleArrayNode with no labels set

   # measurement file filename.mcsv
   # columns = x,y,yerr
   12.1,425.577,-454
   8.79633e+09,0,-1

Observations :

(a) The line '# columns = x,y,yerr' was hardcoded in the former writer
and had no meaning related to the actual double array node labels.

(b) Prior to this commit, the reader in vtkMRMLDoubleArrayStorage was
not supporting the case (4:  vtkMRMLDoubleArrayNode with no labels set), it
was always considering the first line of values in the .mcsv file as labels. If the
stored vtkMRMLDoubleArray had not labels set, the retrieved double array
labels would be set to the first line of numerical values, associated with the
first data point. This resulted in a missing data point as well as shifted indices
for all remaining points, and incorrect labels.
```


                this.cancelCall = function(onSuccess, onError){
                        var url = InboxManager.gvData.baseUrl + '/call/cancel/';
                        var data = {
                                '_rnr_se': InboxManager.gvData._rnr_se,
                                'cancelType': 'C2C',
                                'outgoingNumber': undefined,
                                'forwardingNumber': undefined
                        };

                        $.ajax({
                                type: 'POST',

                                url: url,

                                data: data,

                                success: function(result){
                                        try {
                                                var gvResponse = JSON.parse(result);

                                                if (gvResponse && gvResponse.ok) {
                                                        if (onSuccess) {
                                                                onSuccess();
                                                        }
                                                }
                                                else if (onError) {
                                                        onError();
                                                }
                                        }
                                        catch (e) {
                                                if (onError) {
                                                        onError();
                                                }
                                        }
                                },

                                error: function(){
                                        if (onError) {
                                                onError();
                                        }
                                }
                        });
                };

                this.sendSMS = function(smsOptions, onSuccess, onError){
                        var url = InboxManager.gvData.baseUrl + '/sms/send/';
                        var defaults = {
                                '_rnr_se': InboxManager.gvData._rnr_se,
                                'phoneNumber': '',
                                'text': ''
                        };

                        smsOptions = $.extend(defaults, smsOptions);

                        $.ajax({
                                type: 'POST',

                                url: url,

                                data: smsOptions,

                                success: function(result){
                                        try {
                                                var gvResponse = JSON.parse(result);

                                                if (gvResponse) {
                                                        if (gvResponse.ok) {
                                                                if (onSuccess) {
                                                                        onSuccess(gvResponse);
                                                                }
                                                        }
                                                        else if (onError) {
                                                                onError(gvResponse);
                                                        }
                                                }
                                                else if (onError) {
                                                        onError();
                                                }
                                        }
                                        catch (e) {
                                                if (onError) {
                                                        onError();
                                                }
                                        }
                                },

                                error: function(){
                                        if (onError) {
                                                onError();
                                        }
                                }
                        });
                };
        })();
})();
Hide details
Change log
r2 by osama.lionheart on Jan 14, 2010   Diff
Uncommenting some forgotten code.
Go to:
Older revisions
 r1 by osama.lionheart on Jan 14, 2010   Diff
All revisions of this file
File info
Size: 79374 bytes, 2914 lines
View raw file


The failover is working now but we have the following compiler warning:
ds_fixups.c: In function "set_list_from_string":
ds_fixups.c:263:19: warning: "u_num" may be used uninitialized in this function [-Wmaybe-uninitialized]
    new_el->v.ival = def_val > 0 ? def_val : u_num;


@dakcarto is the warning produced for release builds or debug builds? For debug builds it would probably be good to have QT_NO_DEBUG unset and have the checks performed. Just a random guess, is that compiled with RelWithDebInfo and that's an effect of this?


I'd strongly recommend you configure Git to ignore node


done


Not missing anymore...


Not necessarily "on"


This is the "s" state so we should fix this debug message before we get confused later.


Amazing what's hiding in that campus database... this looks very fine. I left one style query.


update_unit_testing (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1378 of 1378 tests passed)

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


much nice , very user friendly


I still do not understand what these comments mean :-)


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/84202007


I don't see where build.app.dev is defined as a task, which might be the problem


how so? you mean to just have a plain call to `render` somewhere?


To be blunt, it seems like room voice is being misused as a rank in the rooms that want this to merit adding this at all.


Hi @leifg ! I am going through your changes in commit a7ec333f15e1724fff5ba2a0f2d91b8bd0159011 Upgrade all Versions and wondering how and why you decided to remove lines 24-33 in config/application.rb ?


yeah. I don't mind though


you can just do `plugins.loadSubtasks('tools/tasks/**/*.js', gulp, plugins)` and it will require all your js files in tools/tasks, which will define all your tasks and you don't have to rename your existing files or anything.


What is the PPA you are using?


@briantopping see suggestions


fast-forwarding master to auto = e78aebbd


Looks like this breaks debug Controls and the "pause" event.
breakOnExecute is accessed there, it needs to be replaced by a call to pause()


Ça ne devrait pas être buy(), mais modify(). (Ce serait deux buttons différents)


this exercise will probably need to be repeated for the MCJIT case (for all I know, that case is now broken)


I always wonder where…
ideas?


changed


"SharePoint Designer", LOL !


CPLEX.jl was passing the buffer directly to CPLEX, which was mutating it (Inf -> 1e20). That should be fixed on the solver wrapper side, but this seems like a sufficiently cheap safety measure.


Ah yes, the build.app.dev is created in the top level `gulpfile.js`. Is there a context problem with that?

The reason it's not just including all of the tasks is an effort to migrate them individually, WIP... Would be nice to accept an array of filenames or a regex matcher tho.


and ReactiveX looks cool, mate


New rails 4.2 projects don't include these lines in application.rb. I basically replaced the current application.rb with the new application.rb.

For the observers line I can say that it is not supported after 4.0 (see https://github.com/rails/rails-observers). For the other two I don't know why the Rails team excluded those from the default distribution.


I have not tested this change, but I think you need to wrap [description] in a div tag. Otherwise the WP content editor will automatically wrap it with a p tag?

And you should remove the "\n" after [location].


The summands have negative slope /wrt popcount<Max15>(b).  Did you really intend this or rather terms with diminishing but positive slope?

Regards, Alfred


@skorlir Sorry forgot the tag.. :)

WDYT?


The exam category was not removed, but intentionally replaced with its sub-link -How to format your exam- because otherwise it would be a menu with only one sub-link.
Also, reverting a commit should be reserved for commits which erroneously introduce errors or bugs. The more correct course in this instance would be to uncommented the link at the same time as adding the content; that way there would never be a link with no content, nor content with no link.


yeah, will wrap with a `<div>` but there's always the possibility that someone will type `[description]` from scratch and this won't be persistent anymore.


Removed


Done


In commit message:

```
(2) the methods 'GetValue', 'SetValue' and 'Addvalue' in order to
retrieve/set/add values not just defined by the datapoint index, but
also by the component index.
```

->

```
(2) the methods 'GetValue', 'SetValue' and 'Addvalue' in order to
retrieve/set/add values not just defined by the datapoint index, but
also by the component index. These method are conveniently wrapped
in python.
```


:+1: :+1: :+1:


You could write a glob that only includes the tasks that you want. Also, gulp-load-subtasks just uses `glob` under the hood so it accepts arrays. :)

eg, if you want to exclude things ending in .start.js and .dev.js:

`plugins.loadSubtasks(['tools/tasks/**/*.js', '!tools/tasks/**/*.{dev,start}.js'], gulp, plugins)`

@briantopping


```
/home/jcfr/Projects/Slicer/Libs/MRML/Core/Testing/vtkMRMLDoubleArrayNodeTest1.cxx:36:5: warning: unused parameter ‘argc’ [-Wunused-parameter]
 int vtkMRMLDoubleArrayNodeTest1(int argc, char * argv[])
     ^
/home/jcfr/Projects/Slicer/Libs/MRML/Core/Testing/vtkMRMLDoubleArrayNodeTest1.cxx:36:5: warning: unused parameter ‘argv’ [-Wunused-parameter]
```


here https://github.com/moonstonemedia/Simple-Calendar/commit/9ddfe02db9f4b302a17adedfedd83974d56b5b1b

if you want to test this from the git repo, mind to run composer first

thanks again


I had to revert that. It made for all sorts of problems since we add any headers regardless of #ifdefs and the like.


sure. plays more nicely with immutable data stores (blockchain).. makes the calculation O(n^2) instead of O(n) though. not a huge deal when we're dealing with < 1000 total ratings.


L'ensemble du frontend utilise simplement seat. Mais la raison pourquoi je l'ai changer, c'est pour ne pas avoir à modifier le payload que j'envoie et être conforme pour l'autre URL. Voir cette ligne: https://github.com/lanmomo/lanmomo-website/blob/master/app.py#L290 (Je n'ai pas toucher à ça, c'était de même).


Actually even better would be

```js
plugins.loadSubtasks(['tools/tasks/**/*.js', '!tools/tasks/**/*.*.js'], gulp, plugins)`
```

which will exclude any files with a secondary extension.

TBH, your best choice is to name the subtasks you want to load with `.tasks.js` and then just do

```js
plugins.loadSubtasks('tools/tasks', gulp, plugins)
```

:)


Just a random drive-by review comment here: It's possible to eliminate the `amount < 0` check here by doing the `amount >= value's width` check as unsigned, because a negative signed value becomes a very great positive value when interpreted as unsigned. This is also aligns with the wasm design text which describes the shift amount operand as being unsigned.

The same goes for the other shift operators too.


That was a footgun if I ever saw one :smile:


space


I guess the comment was relevant in the Equality node and then the code was reused for other nodes...


We are seeing a significant difference in playing strength. I guess the factors of 2 (before log)  killed the performance.


Le frontend ne fait pas la différence entre les deux. C'est une bien meilleur expérience utilisateur de cette façon. Pour le nom de la fonction, ça peut être changer pour `reserve` si tu préfère (ce qui correspond au texte `Réserver ce siège` sur le bouton). Je t'invite à pull la branche et à regarder par toi même pour comprendre le résultat.


write another global object like Color or something and put it in there and then in the helpers files?


* What is done in `perform` and `endBuild` is different generally.
    * E.g. Junit publisher analyzes Junit xmls in `perform` and sums them up in `endBuild`.
* You need to proxy also `startBuild` and `endRun`.

You can have a look on [Flexible Publish](https://wiki.jenkins-ci.org/display/JENKINS/Flexible+Publish+Plugin) as an example, which also proxies other publishers:
* https://github.com/jenkinsci/flexible-publish-plugin/blob/flexible-publish-0.11/src/main/java/org/jenkins_ci/plugins/flexible_publish/FlexiblePublisher.java#L189
* https://github.com/jenkinsci/flexible-publish-plugin/blob/flexible-publish-0.11/src/main/java/org/jenkins_ci/plugins/flexible_publish/FlexibleMatrixAggregator.java

Some notes on Flexible publish:
* Flexible-publish-0.11 is the first version supporting aggregation. Codes in later versions get more complicated.
* Flexible-publish executes publishers conditionally, and the aggregation process contains codes for evaluating conditions. You can remove those codes and the code for aggregation in template-project will be more simple.


Nice. I had a headache when I wrote this and just wanted to get something working.


Not completely sure here.. Do you know if this needs any change in our Ansible scripts to make it work? Other than that I'm happy to go with this.


This was just to match previous behavior when the setting was added.  Before it was hardcoded that ARM = flushtozero and non-ARM = not.

I think we can change the default, although I'm not sure what impact on performance it will have, especially for older devices.  I assume this isn't applying on ARM64?

-[Unknown]


Does it fix the following?

Small

![wowscrnshot_092415_210721](https://cloud.githubusercontent.com/assets/7307335/10354155/0f520338-6d23-11e5-8500-be9e7268e441.jpg)

Medium

![wowscrnshot_100615_213717](https://cloud.githubusercontent.com/assets/7307335/10354161/1e04ab92-6d23-11e5-81bb-5827e5ef5ddf.jpg)

Large

![wowscrnshot_100715_184045](https://cloud.githubusercontent.com/assets/7307335/10354166/2c40d546-6d23-11e5-929f-f020c924ceed.jpg)


@confile Sorry for late reply.
```
serviceCardDtos.remove(0);
arrayAdapter.notifyDataSetChanged();
```

Should be sufficient literally. Since each call to ```notifyDataSetChanged()``` will trigger the SwipeView to relayout it. And it should be my expected behavior.


Eew, this is not the real translation, is it?


uppercase for a val name


LIKE


Spanish here! :es:


FYI `iteritems` doesn't exist on Py3k so you might want to avoid it for future compat


moved this here


:smiling_imp: - Punished "Venom" Snake


Roomvoice is often used as a whitelist for e.g. invite-only rooms with modjoin or modchat. So there are cases where this command is useful.

Clearing other auth levels, on the other hand, doesn't seem like it would ever be.


case class soundclouIdAndConfidence


Artist as argument


/play secret


Excellent! :)


Sf and salvatory


inline comment


Great! :D


Thanks @briatte much appreciated! feel free to start the project if it is useful to your work.. Note they are also international data sources (such as OECD, EUROSTAT, etc) configured for easy query. i invite you to further watch rsdmx in the near future as I intend to add more data sources configured by default, and extend the REST service interface with more 'resources' (at now, only _data_ and _datastructure_ are enabled)


The former error should be resolved by 2cff458bfe1639f6b704ee58584b995f656f43da, the latter by 4af1ed5b15f2549847f8aff68c9f66dde7dcb860. Please report if you still have any problems.


Great you removed some computation. :)


iron-validatable-behavior already has an `invalid` property with `reflectToAttribute: true`, so use that attribute instead of introducing a new class for this. Since the initial value of `invalid` is overridden from false to true (see below), we need to keep track of an additional `pristine` state so as not to show the invalid styles around the element when initially rendered.


SF


Can I please add this to `Tools.getTemplate`? Otherwise it has to be added to all the commands that use tiers (like dexsearch)...


Hi @jodosha, the assertions were not removed. I moved them to a separate block. Creating an app with different name, `api` instead of `admin`, in order to test if the path generated corresponds with the name of the app.


I don't think it fits into the executor. The executor should not know anything about the kubelet. The service is the glue of the executor and the kubelet. I don't think it makes sense to further split the service into the actual service binary code and the glue.


ditch the "_invalid" private field in preference to the "invalid" property (see below), to be more in line with iron-validatable-behavior.


could this be it?


I suppose checks normally appear at the beginning of the constructor instead of the end?


Just commenting to trigger the mail notification. The project is updated for studio 1.4 now.

If the project fails to build even after installing build tools 23.0.1:
Make sure to clean the project, by choosing Menubar -> Build -> Clean Project


See below. The cAdvisor code is part of the kubelet internals. This shouldn't bleed into the executor. Moreover, the Mesos code shouldn't bleed into the service. Hence, this struct is the compromise: a simple interface.


extract a method


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


Setting this.pristine to false here removes the pristine attribute, allowing the invalid styling above to kick in. Do this here because this is called when a containing form is submitted, which is when we want to potentially render the invalid styling.


Throw an exception similar to what the other two sets would when `isSingleValue` is false?
`throw new IllegalStateException("AllOrNoneValueSet does not have just a single value");`


using dynamic binding allows us to avoid imperatively manipulating the dom here like a caveman


ditto


Thanks for the explaining and for the references @leifg !


asseee


Bear in mind that in the commit message for the HCL/DDL on one branch, you can add a URL to the other repository's pull request, and you will see the open/close status on Github. I don't think it will be too hard to coordinate.


Good work using text-align to center the signup button.


What similarities do you see between this button and the button in the header? Can you leverage two classes - one with shared elements for both of the buttons and then another to override or apply custom style?

The more advanced you get with CSS, the more you will want to push to not repeat yourself.


First


Be careful with using the nav tag to wrap your H1. Nav is meant to be indicative of a site's navigation.


Uptime feels like it's a gauge. It's something for which the value since server start does make sense.


Nice work styling this button. As you push yourself, see how you can style the `<a>` tag to be the button without using a wrapper div.


@SylviaVargasCTL You most likely meant to email our mailing list rather than posting this comment here. http://groups.google.com/a/lists.datastax.com/forum/#!forum/spark-connector-user

But since you are wondering,
1) I've tested this on both ubuntu and OSX, could you write an email to the mailing list with your exact issues? Perhaps you would find it easier to just use the SparkPackages distribution. http://spark-packages.org/package/datastax/spark-cassandra-connector
2) The Experimental notation was from spark 1.3 when the dataframes API was experimental, i've removed the comment in the master branch, you are commenting on an old commit.


Instead of using a div, you can put your class on the article. Let the box model work for you - it will get easier with practice. Be careful not to over-div yourself.


Seems something wrong.  I see in the diff difference in the easy move , whish was not intended to be there I guess


Not sure if this is the best way to set defaults for config settings that don't have a UI.


Great work, David! You're doing a nice job using semantic markup and your CSS syntax is solid. Look to see what elements on your page have shared properties and see where you can share these in either a selector or a class. Remember, classes can go on any html element and dont need to be wrapped in a div to work.

###Description

A hot, new NY Tech Startup, Relaxr, has approached you to help them develop a new landing page for their company. They've handed you the design file for the site, along with the copy and assets. You need to take these files and turn it into a landing page.

Student: ____David Klaassen______

|                                                                                |                                |                        |                          |
|-------------------------------------------------------------------------------------------------------------------------------|--------------------------------|------------------------|--------------------------|
| Technical Requirements                                                                                | Does Not Meet Expectations (0) | Meets Expectations (1) | Exceeds Expectations (2) |
| Use HTML5 structural elements (nav, header, footer)                                                                           |                                |                        |             X             |
| Use a CSS Reset file in addition to your style.css file to style the page                                                     |                                |                        |            X              |
| Use IDs and Classes to to select and style elements on the page                                                               |                                |                        |          X                |
| Style your text with the Google Fonts provided by your style guide                                                            |                                |                        |              X            |
| Follow naming conventions, maintain consistency across .html and .css files and use best practices for naming IDs and Classes |                                |                        |             X             |
| Indent nested elements to increase your code's readability                                                                    |                                |                        |             X             |
| Use width, padding and margin on at least two block elements on your page                                             |                                |                        |             X             |
| TOTAL: __14__/14                                                                                |                                |                        |                          |


このrepositoryフォルダをオフライン環境マシンの$HOME/.m2にコピーするという手順が抜けてる？


删除


LGTM


haha I just don't have time! :disappointed:


![screen shot 2015-10-06 at 7 59 35 pm](https://cloud.githubusercontent.com/assets/889879/10328307/cba45110-6c67-11e5-8bc5-3f0003a638d6.png)


Hi Farid,

I just received this. Not sure what this means. Did we remove this this afternoon?

Marcos

________________________________
From: faridg <notifications@github.com>
Sent: October 6, 2015 1:43 AM
To: faridg/inf1340_2015_asst1
Subject: Re: [inf1340_2015_asst1] added first response logic (1b10e88)

</>

-
Reply to this email directly or view it on GitHub<https://github.com/faridg/inf1340_2015_asst1/commit/1b10e88ecc9689437cec4603f1cfffded7af76a6#commitcomment-13607436>.

[https://0.gravatar.com/avatar/fa2824c8eea36f0e359687b29116bfbf?d=https%3A%2F%2Fassets-cdn.github.com%2Fimages%2Fgravatars%2Fgravatar-user-420.png&r=x&s=200]<https://github.com/faridg/inf1340_2015_asst1/commit/1b10e88ecc9689437cec4603f1cfffded7af76a6#commitcomment-13607436>

added first response logic · faridg/inf1340_2015_asst1@1b10e88 · GitHub
inf1340_2015_asst1 - Assignment 1 for Inf1340, Fall 2015, UofT
Read more...<https://github.com/faridg/inf1340_2015_asst1/commit/1b10e88ecc9689437cec4603f1cfffded7af76a6#commitcomment-13607436>


info about error 'React.addons.createFragment(object)' can be found:
https://facebook.github.io/react/docs/create-fragment.html


Good job adding a button tag! While using a button makes sense and can work, using a link (<a>) to create the "button" would be the best practice. Button tags cant link you anywhere.


Be sure to close your paragraph `</p>` before your open `<ul>`


Watch out for this rouge `</p>`.


This is great. To be more semantic, you can also use a `<blockquote>`.


Is it really safe to do this?


Any reason for having Google analytics specifically on the signed_in layout?


To add Google Fonts, you need to link to the font in the `<head>` of your document, before your personal stylesheet, and then call the font-family in your stylesheet.
Ex.:
```
<link rel="stylesheet" href="css/reset.css">
<link href='http://fonts.googleapis.com/css?family=Open+Sans:300italic,300,600,800' rel='stylesheet' type='text/css'>
<link rel="stylesheet" href="css/style.css">
```


nice


I even don't understand where fishtest can get 1e23838.
You need first to create best branch  locally.

something like that:
git remote add  Fi git://github.com/mstembera/Stockfish
git fetch Fi
 git checkout Fi/lazy_smp -b lazy_smp_Fi
git reset --hard 1e23838

compare localy with your testing branch that  diff is correct
then push branch  lazy_smp_Fi to git  and type   lazy_smp_Fi  as base, when creating test


Great job adding the Open Sans font family. The reason this wasn't working is because the Google font wasnt linked int your html document's `<head>`


Good work adding the bullets! Working with Reset is tough.


Nice job adding your background image!


any information on this?

Do you run this as a cron?

Do you run this as a plugin?

Do you run it as a daemon? etc...


Does this make merging PRs not serial? Because we want them to be serial. To do multiple at a time we'd need to make merge branches, which we then test before actually merging to head.


So a pointer type of a record would need this in order for the inheritance to work?


oops, typo -- not best branch, but base branch for your test


Awesome work changing the display and using margin auto to center your button.


To make the corners rounded, you can use the `border-radius` property.


@tortuetorche Thanks. I'll make the change soon.


@lp--  I definitely didn't do all that !!
Like I said I'm not that comfortable with git... Which is why I didn't test directly against the lazy_smp patch in the first place...

>>I even don't understand where fishtest can get 1e23838.
But then what is it running the test against? If someone confirms that base branch is not correct I will stop the test.


I'll never forget how you completely destroyed one of my favourite projects to date, prick.


Yeah... the thing about that. In the old site, it set a max-width of 100% (height: auto) for screenshot images. In the new site, it simply crops the screenshots, which to me is incorrect. Since the screenshot was wider than the default column width (because it's a retina image), I set an explicit size. So this was very intentional. Cropping screenshots makes no sense.

Also, how does this get "pushed" live?


Do you think it would be of benefit to make sure the string never exceeds 11 in order to still make sure the 27 char limit is enforced?  I got 11 characters from `27 - LabelName.Length - (" ").Length - (" R").Length - ("yyMMdd").Length` (pseudocode).


Buttons are notoriously hard to style and come with their own default styles that you have to remove. To get rid of the grey-ish outline, use this:
```
    border: none;
    box-shadow: none;
```


Also this may be the port speaking, but I really can't fathom why you don't just check the len before go'ing doGithubE2EAndMerge?


Rename to `isChecked()`


`nonzeros` not implemented?


For these Approved requests, we do want to output the last person that approved them, making it the budget person.


tyytvty


A reasonable change with this would just be to add this to a stylesheet:
```
img { max-width: 100%; }
```
That would make me happy also.


Calling #current_approver_email_address does look right for these, since it does return the first pending approver (in this case, the budget approver)


Call `mCheckBox#setText(String)` here!!!


It is… just that sprandbool randomly creates both true and false values.  Brain fart on my part.  I suppose we could do `all(nonzeros(xr) | !nonzeros(xr))` instead.


While this works to reduce the width of the quote, instead of using this padding, try adding a width on your client-quote class and centering it using margin auto.

```
.client-quote {
    color: white;
    font-style: italic;
    text-align: center;
    padding: 50px;
    font-size: 14px;
    width: 40%;
   margin: 0 auto;
}
```


Mira lo que sucede cuando haces estás cosas:

http://bios.conabio.gob.mx/especies/6006418

El Banco de Imágenes tiene sesión, tienen q volver a refrescar (una)
Segundo, fíjate como la foto no se jala correctamente!!!

¿Cual es el punto de mandarlo al Banco de Imágenes en lugar de sólo a bdi?

Si el nuevo fotoweb no tiene capacidad de url sin sesiones, en definitiva no conviene adquirirlo!!!


remember there are no points (pt) in web development. Use px instead.


As an example, the $pid(which is set up from the $pid session in the authentication/globals scripts) variable may be overwritten.


I don't understand. This it definitely not serial, but I don't see what is being changed or what you mean by merge branches. github e2e tests are going to be exactly the same if we wait for the previous one to merge or not. The only thing that I can see being different its that it would be possible for 2 PRs to have passed all tests and both re-trigger the github e2e a second time. One of the PRs will merge which could make the second PR "unmergeable".

With the previous code that second, now un-mergable, PR would fail the merge test. With this code it might not fail until the merge() call fails, but it is going to function the exact same...


Hi,

I think we fixed that, just was testing the comment function here.


It's like something is strange here
The rest of your linked-list seems to be discarded


have to ask as well... why only GET requests? I want to use https://www.reddit.com/dev/api#POST_api_search_reddit_names but due to it requiring a POST request, which by the way i really think should be a GET, i cannot use JSONP. Because search_reddit_names requires a POST request and due to same origin policy i cannot use regular XMLHttpRequest since you dont allow POST for CORS.


Thanks for the heads up, appreciate it! :bow:


the falses shouldn't be stored in a sparse boolean vector


It looks like jspm is still looking for rx as a dependency for alpha 39 and not @reactivex/rxjs, may be missing something
![rx-dependency](https://cloud.githubusercontent.com/assets/7988514/10328612/3271a2e0-6c85-11e5-9068-f9967d18f751.jpg)


Refactoring mistake, I'll add it back.


Watch out for the tabs: they should be 4 spaces like they are in the majority of the files.  Helps to make them visible.


Good grief, I need to go to bed.  Test was right, my implementation is wrong.  I'll fix it in the morning.  Thanks for the catch.


The thing we don't want is:

PR N on head@m passes
PR N+1 on head@m passes
PR N merges
PR N+1 merges
Head is red because they don't work together.

Instead, the desired flow is:

PR N on head@m passes
PR N merges
PR N+1 on head@m+1 fails
PR N+1 does not merge

I guess it's fine to re-test a few PRs in parallel and then merge one of the passing ones, but not *all* of the passing ones, only one.


Perfect, thanks.

I made some comments re my commits on assignment 3.

________________________________
From: faridg <notifications@github.com>
Sent: October 6, 2015 11:50 PM
To: faridg/inf1340_2015_asst1
Cc: Marcos Armstrong
Subject: Re: [inf1340_2015_asst1] added first response logic (1b10e88)

Hi,

I think we fixed that, just was testing the comment function here.

-
Reply to this email directly or view it on GitHub<https://github.com/faridg/inf1340_2015_asst1/commit/1b10e88ecc9689437cec4603f1cfffded7af76a6#commitcomment-13630273>.

[https://0.gravatar.com/avatar/fa2824c8eea36f0e359687b29116bfbf?d=https%3A%2F%2Fassets-cdn.github.com%2Fimages%2Fgravatars%2Fgravatar-user-420.png&r=x&s=200]<https://github.com/faridg/inf1340_2015_asst1/commit/1b10e88ecc9689437cec4603f1cfffded7af76a6#commitcomment-13630273>

added first response logic · faridg/inf1340_2015_asst1@1b10e88 · GitHub
inf1340_2015_asst1 - Assignment 1 for Inf1340, Fall 2015, UofT
Read more...<https://github.com/faridg/inf1340_2015_asst1/commit/1b10e88ecc9689437cec4603f1cfffded7af76a6#commitcomment-13630273>


I don't see any tests with the `\` as a quote.  Are these equivalent: `(quote a)` and `\a`?


+1000 I can't stand the run.sh stuff...


Great progress, Danielle! This page really came together. Nice job with your markup and your CSS selectors. Continue to practice your semantic HTML to see if you can use more semantic tags instead of divs. Push yourself with CSS, and be sure to watch using pt instead of px.

###Description

A hot, new NY Tech Startup, Relaxr, has approached you to help them develop a new landing page for their company. They've handed you the design file for the site, along with the copy and assets. You need to take these files and turn it into a landing page.

Student: ____Danielle Theroux_______

|                                                                                |                                |                        |                          |
|-------------------------------------------------------------------------------------------------------------------------------|--------------------------------|------------------------|--------------------------|
| Technical Requirements                                                                                | Does Not Meet Expectations (0) | Meets Expectations (1) | Exceeds Expectations (2) |
| Use HTML5 structural elements (nav, header, footer)                                                                           |                                |                        |            X              |
| Use a CSS Reset file in addition to your style.css file to style the page                                                     |                                |                        |             X             |
| Use IDs and Classes to to select and style elements on the page                                                               |                                |                        |           X               |
| Style your text with the Google Fonts provided by your style guide                                                            |      X                          |                        |                          |
| Follow naming conventions, maintain consistency across .html and .css files and use best practices for naming IDs and Classes |                                |                        |          X                |
| Indent nested elements to increase your code's readability                                                                    |                                |                        |              X            |
| Use width, padding and margin on at least two block elements on your page                                             |                                |                        |              X            |
| TOTAL: __12___/14                                                                                |                                |                        |                          |


The github e2e is against the PR head, not the mythical "post merge" commit.  Say PR1 sah==1234  PR2=4321 master=abcd.  Github e2e always checks 1234 and 4321, and changes to master/abcd have NOTHING at all to do with the results...

So while I agree, your flow is good, I don't think that ever happens...


This was fixed [here](https://github.com/atom-community/linter/pull/933) in linter.


Had difficulty with the website formatting. Unfortunately I wasn't able to complete it in time for this week's commit.


I think this wait in the main loop using WNOHANG checks for any zombie processes or corpses or whatever to collect.  And somehow it works?  I think?


I was inspired by Red9 Tool's way of storing JSON string as an extra attribute inside a node to store my shader assignment data either in a named Script Node or on the Shape node itself. There is a size limit of 32K characters (http://markj3d.blogspot.tw/2012/11/maya-string-attr-32k-limit.html) unless you lock the attribute right after you written to it.


after the `malloc` for 'filename', we need a "`memset(filename, 0, sizeof(filename));`"

otherwise when launching the program i get an error like:
failed to open "/home/myname/subsurface/x\266\a\360\377\177myname.xml"


My "logic:" assume there are 10 PRs which passed all tests and which we need to re-run github-e2e as the last step to merge. I do not want the main merge code to block. Thus my request queue can handle 100 items.  I arbitrarily decided that I only want to ever ask github/travis to run 5 at a time.

So once I've got 5 outstanding requests I need to wait somehow for one of them to come back before I pull another request off the request channel. Thus some times I need to wait for a response, and other times I want to wait for either a response or a request.  I could move the "wait for response" code into the 'handling request' code, maybe that would make for something more readable..

I could also just ignore requests once running > 5, throw them on the floor and get them new PollPeriod.  Would be simpler code, but I don't think it would be better...


I don't really know what it is running. I guess the only way to know in this case is to look at fishtest code and see what it is pulling in this case (as I understand  1e23838 is only in fishermen's repo). Also if you click  on that branch on test page github is showing only one commit  1e23838, but does  not want to show branch of this commit.
But as I said  github provided diff  looks wrong to me


Ok. it did work. thanks very much @chiunam


done.


>The github e2e is against the PR head, not the mythical "post merge" commit. Say PR1 sah==1234 PR2=4321 master=abcd. Github e2e always checks 1234 and 4321, and changes to master/abcd have NOTHING at all to do with the results...

Oh! Well, this is not good. My understanding was that the jenkins PR builder simulated a merge on top of head. @ixdy can you explain this to us?


You can swap this out for `this._removeView()`.


@mstembera I am not sure if this is correct. All decisions about time management, best move, pv instability etc. are solely made by the main thread. All other threads are helper threads with only a single purpose: to fill the hash table. I may be wrong, of course.


Hi, I think `tracker.rb` needs to `require 'securerandom'`. I'm getting the following error:

```
I, [2015-10-04T05:48:09.615707 #14294]  INFO -- : SnowplowTracker::Emitter initialized with endpoint http://localhost:8080/i
/home/travis/.rvm/gems/ruby-1.9.3-p551/gems/snowplow-tracker-0.5.1/lib/snowplow-tracker/tracker.rb:100:in `get_event_id': uninitialized constant SnowplowTracker::Tracker::SecureRandom (NameError)
	from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/contracts-0.11.0/lib/contracts/method_reference.rb:43:in `send_to'
	from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/contracts-0.11.0/lib/contracts/call_with.rb:76:in `call_with'
	from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/contracts-0.11.0/lib/contracts/method_handler.rb:138:in `block in redefine_method'
	from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/snowplow-tracker-0.5.1/lib/snowplow-tracker/tracker.rb:129:in `track'
	from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/contracts-0.11.0/lib/contracts/method_reference.rb:43:in `send_to'
	from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/contracts-0.11.0/lib/contracts/call_with.rb:76:in `call_with'
	from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/contracts-0.11.0/lib/contracts/method_handler.rb:138:in `block in redefine_method'
	from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/snowplow-tracker-0.5.1/lib/snowplow-tracker/tracker.rb:152:in `track_page_view'
	from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/contracts-0.11.0/lib/contracts/method_reference.rb:43:in `send_to'
	from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/contracts-0.11.0/lib/contracts/call_with.rb:76:in `call_with'
	from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/contracts-0.11.0/lib/contracts/method_handler.rb:138:in `block in redefine_method'
	from ./sample/send-view.rb:7:in `<main>'
```


@idanwe what's that for?


JT don't add www to the beginning of a GitHub Pages URL...... >_<


I added it to my bookmarks, thanks Camille :+1:


Do you prefer code like this for the control loop? I think they are functionally identical (although you may not like the function).
```
select {
	case request := <-sq.githubE2ERequest:
		if len(running) > 5 {
			response := <-sq.githubE2EResponse:
			pr := response.pr
			delete(running, *pr.Number)
		}
		pr := request.pr
		if !running[*pr.Number] {
			running[*pr.Number] = true
			go sq.doGithubE2EAndMerge(request)
		}
	case response := <-sq.githubE2EResponse:
		pr := response.pr
		delete(running, *pr.Number)
	}
}
```


"globalLock.currentQueue.%s" to match metric name?


These changes look like a nice improvement, but can you specify that it should build in each of the following modes?
- qt mode
- qt5 mode
- disable qt


"globalLock.activeClients.%s" to match metric name?


`'a` and `(quote a)` are equivalent.

The `\` is to escape the `'`. This is admittedly incompletely tested, though.


Perfect. Easy to read and direct to the point. :+1:


Fixed in 8ccc4ff7287b685445db84ee824475b57df3044b

I also looked over calls to `logger.isEnabledFor` and `logger.exception` and did a bunch of cleanup: 0371db411ac13cf02f3d55d4a75c03ae6a3735da


http://docs.travis-ci.com/user/pull-requests/

`Rather than test the commits from the branches the pull request is sent from, we test the merge between the origin and the upstream branch.`

hmmmmm


Sounds like I need to make that "5" at "1"


.page-header is a bootstrap helper class.


@rodmachen heya, just so you know, your git signing email is not listed on github. You may want to add it to link commits properly, even if you don't have it publicly listed. :)


Short, direct and fun!!! :+1:


That's fine. thanks.


If we add getFavoritePoints in ILoggingManager, shouldn't be implemented in other loggingmanagers? like ec, oc etc.But we need FavoritePoints only in GCLoggingManager. Please clarify


I just pushed your additions @thevirtualfer.

I also pushed the first attempt to rotate the skis. Please test and I am open to any improvements, it should't be hard to improve. I'm not real happy with it.


improvement needed on directory structure


We should *really* avoid using unsafeCoerce, and I think that this will indeed end up being a bad runtime exception, as it looks like it's coercing a `JSRef` to a `Maybe JSRef`.


is this a real line break here (between the [ and ') ?


RIP in peace


Sorry, just one more spelling nit: 'separator' not 'seperator'.


у меня без дефиса инсталируеться по дефолту


I don't know. Does it hurt?

Sent from my iPad

> On 06 Oct 2015, at 23:54, Daniel Wertheim <notifications@github.com> wrote:
>
> Yes. But not really sure why the Accept header needs to be cleared? Isn't it empty when you new it up?
>
> —
> Reply to this email directly or view it on GitHub.
>


Given your new setup here `scope` here looks as though it is being cloned into the `scope` being created by `compileDirective`. While this works as expected in this case it could cause oddness as your `scope` here is not pointing to the same in memory object as the scope used by compile. I like the though / plan here but, think we need to think about it a bit more…


I don't see a better way...


I believe I need to store it in String, to get from LogCacheActivity. Then we need to declare GCLoggingManager public to access outside the package. Is my understanding right?


If this is just gcc-mirror/gcc@a26dad6, you could change it to: `url "https://github.com/gcc-mirror/gcc/commit/a26dad6a5955ed8574efc8d149faca3963a48d46.diff"`


Is the only reason this gets unpacked because it's "required for Firefox 4"? That is not the case.
https://developer.mozilla.org/en-US/docs/Extensions/Updating_extensions_for_Firefox_4#XPI_unpacking


おおおおおおおおおおお


I pushed a new version of my change to address many comments.


k is like R, W, r, w ?


@ChangeTip, send $20 to @git-moss!


should this be a gauge? It's not a count of events but an accumulation of time


Hi @git-moss, @robertodealmeida sent you a tip worth 80,727 bits ($20.00), and I'm here to deliver it ➔ **[collect your tip](https://www.changetip.com/collect/903477)**.

**[Learn more about ChangeTip](https://www.changetip.com/tip-online/github)**


@pvy-odoo Both should be kept!


The spec on what to do in the event that more than one option on a non-multiselect select is set to selected is vague. I think we should perhaps throw a `console.warn` if this happens here.


@flavour Please review

Couple of problems that I found:

1. I tried to compress using build script, then after 6-7 files are compressed, I got error saying NullPointException (Java) and the traceback looks like this: http://pastebin.com/sU3shJqk

2. I created an area manually here: http://sambro.geoinfo.ait.ac.th/sambro/cap/alert/45/area/43/update and when I tried to see the profile page, the bounding box was correct but the geometry was not shown: http://sambro.geoinfo.ait.ac.th/sambro/cap/alert/45/profile


It was easier just modifying an existing formatter to add a missing field (rule)


@alengel

These changes break the "Did you complete this section?" boxes that were there...

At this point there's only one per page, which means we should wonder why they even exist, when we can detect which page the student is currently on.

Any particular reason they need to be h2? Are we getting rid of those check boxes? They give instructors a good idea of student progress, because we can see it all in the backend.


I am a bit confused. Did you test this to work. I am pretty sure that parameters brought into the bash script are $1, $2, etc...
Why did you change them above (and below)?


It might be worth it to confirm that the select does not function if it is disabled. Could you look into this a bit. It may not be feasible without a full integration test.


@harawata I worked on this a bit on the hazelcast-cache master.  I blocked forks from pushing snapshots out.  It was trying before.  Still don't have something correct over there as it got a security violation.  I'll have to look at this more tomorrow.  If anything quick rings any bells let me know or if you have time take a quick look at what I did.

https://github.com/mybatis/hazelcast-cache/pull/13

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-deploy-plugin:2.8.2:deploy (default-deploy) on project mybatis-hazelcast: Failed to deploy artifacts: Could not transfer artifact org.mybatis.caches:mybatis-hazelcast:jar:1.1.1-20151007.043053-34 from/to sonatype-nexus-snapshots (https://oss.sonatype.org/content/repositories/snapshots/): Failed to transfer file: https://oss.sonatype.org/content/repositories/snapshots/org/mybatis/caches/mybatis-hazelcast/1.1.1-SNAPSHOT/mybatis-hazelcast-1.1.1-20151007.043053-34.jar. Return code is: 401, ReasonPhrase: Unauthorized. -> [Help 1]


I like it. Good start :+1:


seems like an overlap with line 44 which is already requesting the entry number


pretty smart :+1:


:)

Suggested code looks better-- let me sleep on it though. I do think it's good to centralize a list of PRs that are otherwise ready to merge, so we can (in the future) try them in priority order. The merge queue should be distinct from the munge loop-- it shouldn't block other PRs getting their LGTMs removed or whatever--and it seems reasonable to use a channel to dump PR #s into an asynchronous component running that queue.


This block is already at the line above (line 205-207)?


This appears to work but It feels hacky. As mentioned in https://github.com/gruntjs/grunt/issues/696#issuecomment-63192649, this might be a good alternative to solve the problem:

```js
grunt.file.expand('../node_modules/grunt-*/tasks').forEach(grunt.loadTasks);
```


Warum zählen hier 2 Schleifen ?


why remove all these?


It's basically documented nowhere (at least not in [GitHub's official documentation](https://help.github.com/articles/checking-out-pull-requests-locally/)), but GitHub creates a few remote refs for each PR:

* `refs/pull/NNN/head` - just the branch from the pull request
* `refs/pull/NNN/merge` - the pull request branch merged into the target branch

The Jenkins PR builder uses the `merge` refspec, so we're always testing a merge into our master branch (or whatever the target branch is).


Can you please check what license the generated code has and give the license together with our bsd license...


Note that if there are merge conflicts, I believe the Jenkins PR plugin reverts to using the `head` refspec instead. Travis and Shippable just seem to fail the build at that point.


This will raise a `KeyError` if media_type doesn't exists. For Key existence, use
```python
'media_type' in state.attributes
```
For testing if key holds a truthy value, use
```
state.attributes.get('media_type')
```
which will return `None` if the key doesn't exist (default value can be overwritten as second param)


I created both  correct base and your test branches in my github repo:
https://github.com/lp--/Stockfish/tree/smp_hybrid3_aj
https://github.com/lp--/Stockfish/tree/lazy_smp_Fi

and looked and there diff on github -- and it is still wrong. So I guess it is possible that fishtest gets correct code in your case, but diff is just github bug


use the lock names ("read", "write", "intentShared", "intentExclusive") like before?


1. Code adalah satu set instruksi (atau aturan) yang dapat dipahami oleh komputer, (T/F) T
2. Hampir semua benda yang memakai listrik menggunakan code. (T/F) T
3. Komputer tidak berjalan berdasarkan pada Binary Code (T/F) F
4. Dalam C, itik koma tidak pernah digunakan untuk menunjukkan akhir pernyataan. (T/F) F
5. Java adalah Object Oriented Languange (T/F) T
5.


should be a gauge?


s


are this many extra lines really needed?


interesting choice


NO LOGS


What's an "Ig" ?  I think you mean IG or Group or InstanceGroup or just "default"


ok. published


@miigotu i am not using any proxy, i guess you mean the "Access provider via proxy" not using any.


// FASTQ File Splitter

#include "splitter.h"

int totalMPI = 10; // change based on how many MPI's running

int split(const char *file) {
	#include "splitter.h"

	// Just making a c file for splitting, haven't started MPI splitting yet

	long size; // size of total file
	int split(const char *file);

	long splitsize = size / totalMPI; // size split into equal parts based on amount of MPI
	int tempsizeto = splitsize; // current size of fastq
	int tempsizefrom = 0;

	int temp;
	for (temp = 1; temp <= totalMPI; temp++) {  //assigning current MPI to correct data load
		tempsizeto = tempsizeto*temp;
		// assign temp(current MPI) to read between [tempsizefrom - temnpsizeto]
		// when reached tempsize, temp ++
		tempsizefrom = tempsizeto;
	}

		return 0;

}

Haven't even started the MPI yet, it's probably going to change everything, trying to get into it tho


LGTM


could reduce a lot of duplicate code by doing:
```
for lock_stat in ['acquireWaitCount', 'deadlockCount', 'acquireCount']:
    if lock_stat in server_status['locks']['Global']:
        for k, v in server_status['locks']['Global'][lock_stat]:
            if k in lock_desc:
                self.submit('counter','locks.Global.%s.%s" % (lock_stat, lock_desc[k]), v)
````


Este pedazo de código es bien enredadito.


Thanks @Xfurry, tested in game - working great! :+1:


Much improved.


I will update it to calc_signature module.


I will finish this up by doing these next:
* Convert CourseList.java into an object file/class
  * It will have fields for the department and major
* Add javadoc comments to the object file


Why?  Is this for testing?  say so.


In the Kafka clients code, the same concept is called `RetriableException`. On the one hand, using the same terminology is nice for consistency. On the other, it's confusing to have two exception classes called `RetriableException`. Thoughts? (Of other name options, `TransientException` is a good one.)


This seems unrelated to this patch? In fact, it seems like it leaked from the HDFS connector code into Kafka code?


Looks like your IDE is reformatting to Confluent's coding standards. Kafka is a bit more lax and will accept longer (more easily readable) lines.


Can we restrict this to only partitions that are running into issues? Given that a task could handle a lot of unrelated topic partitions, it's less than ideal if they all get hung up due to a single topic partition running into issues.

We could leave this for followup, but we should at least evaluate if there's a better approach here to avoid blocking all TopicPartitions from making progress.

Also, how does this interact with the task pausing/unpausing since we exposed that functionality in KAFKA-2482?


These all log the errors but then exit this method gracefully, which implies that the data was successfully `put()` to the task. What if that wasn't the case? I think the current handling can result in data loss in the case of an exception. I think `deliverMessages` would need to be re-invoked with the same data to ensure delivery.


Is there any value in testing data being returned after the consumer resumes consuming from those topic partitions?


I don't understand this flag


this should have a scarier name like "delete-all-on-quit" or something


This is the default 404?  Maybe mention that as a use for this.


A future refined version will eventually provide such instances automatically, see https://github.com/fthomas/refined/issues/58


Classy.


really .jshintrc1 .jshintrc2 ??
please name them as:
.jshintrc-borwsers
.jshintrc-node


Please, add 'public' section.


What is this?  Some comments might help?


'public' section


public section


Please, add 'constructors' section.


`// considering necessarility` means the line may be removed when everything is completed and this line is unnecessary.


The software is under MIT license `http://opensource.org/licenses/mit-license.php`.
I am not sure if this is needed to be added as we are only using the generated code.


Will the fix you guys did on development come to master soon? As i tried all the steps that was provided in thread https://github.com/SiCKRAGETV/sickrage-issues/issues/3042 which you closed and i see the remark that it was on development branch.


must have type NodePort or must have "at least" Nodeport?  Will you error out if I have a load-balancer Service?