把cookie换成session。
另外，这个位置也不对，没有判断$result是什么情况就做存储操作是不合逻辑的。


`recordNodesIter` -> `nodeIterator`


I'm as much in favour of ending files with a (single) newline as you are, and somewhat rigorously enforce it in projects I maintain. Sorry that the commit message didn't give the full context on the change; it has to do with doing one thing alone, doing it well – and doing the right thing at the right level of the stack.

The cli tool you'd use from the shell adds a trailing newline (because `console.log` does), so this utility function itself shouldn't add another one, as one nl-at-EOF is good, but two isn't:

![newline-at-eof](https://cloud.githubusercontent.com/assets/2459/6528886/58ce48de-c425-11e4-81ea-781c08ded76b.png)

The last line is an example of what happens if `require('format-json').diffy` thinks it is the last end of the chain (it isn't), adding policy whitespace outside of the actual json itself. If the node `console.log` had returned a byte count, it would have said four here: two braces, and then two newlines.


I can't see any evidence that the -aux switch still exists, other than the fact its included in the usage message. If that's really the case then obviously the usage message should be changed.


@joshnuss Would you mind pulling and having a look at this when you have a second? I've been trying to keep the front-end upgrade branch in sync with master, but had to revert the product model to its previous state in order for the variant dropdowns to keep working.
I am also seeing the following error on the console (might be related):
![screen shot 2015-03-06 at 11 24 48 am](https://cloud.githubusercontent.com/assets/865198/6528941/7827eaaa-c3f3-11e4-9b2e-3fe79741a14f.png)

Don't hesitate to commit changes to this branch if necessary.


r+


Done


使用框架的信息采集方法。
该处有注入漏洞。


hehehe I made some changes on previous BZ in order to make the FMPP extend KieEditor and that change made everything easy!


I think negative numbers are worth mentioning it. A user could also expect that the function throws an exception when providing such values.
Your description is better, but I would it extend by:
"Providing a negative number performs no splitting."


@RubenSandwich Given the fact that you are the one who wrote:

Add or Pick a task and mark it if your working on it.

I'm not sure why you unchecked that, as you are working on it(side note, it should be 'you're working on it')


Done with https://github.com/TYPO3-Documentation/TYPO3CMS-Reference-Typoscript/commit/68689b5101f5cc32f154e704524fb49221d85804


@ipgray Can't do this one, I need the institution


Thought so... will remove


If it makes more sense, we can include the `preprocess` (or `src`) directory within `assets` and tell the CLI tool to ignore that file as described [here](https://github.com/Shopify/shopify_theme#the-configyml-file). Perhaps we should include a sample `config.yaml` anyway, which can include suggested files to ignore.


@ipgray Can't do this one, I need this institution


Good question. Let's change that.


RESULT=success there will cause any later test failure to be ignored.

RESULT=success should only be set at the end of main.sh


Sorry for the blame @PolygonalTree, but this function **should have** an `id` argument. Also, it it does not use the `device_info` arg. See #59


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


When I think about it it's relly better, since the splitting functions would crash then (they expect a list, not a string).


Ah, true, yay for copying and pasting.


Good job, thanks!
![image](https://cloud.githubusercontent.com/assets/2530471/6529048/8dbcfdd2-c43f-11e4-8bb3-552717d15571.png)


We really should do that. `re.finditer()` really returns an iterator, but if providing `max_matches > 0`, then we return a list. That's somehow inconsistent.


Done with https://github.com/TYPO3-Documentation/TYPO3CMS-Reference-Typoscript/commit/c9c236061e8b402661d2a84a3bd011fd28295db6


K, I'm working on that.


Then each documentation has to be rewritten to yield an exception for negative numbers^^


30411/Sporis A. Ioan Catalin/Assignment 1.txt

I see no reason why this should fail. I tried it locally and it works


Is this formulation right?  It seems to me that the workerLoop will now run "hot" if there are no messages, going around and around until CurrentQueueLength > 0 or TryDequeue works.  Won't that burn CPU on the worker?


I'd create a whole new op for this


I guess you forgot an `index=True` there?


Ah. If @sfgraves recently chopped it out then, yes, it should be expunged from the usage section.


Will this fix be included in a release any time soon? I have upgraded from 0.0.2 to 0.1.0 but the problem still persists. It appears that this fix wasn't included in that release.


Really? Have you a good name then?


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

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


Done with https://github.com/TYPO3-Documentation/TYPO3CMS-Reference-Typoscript/commit/68689b5101f5cc32f154e704524fb49221d85804


Is this a valid conversion of the test case? Wouldn't Ops b loaded more eagerly if it is referenced like this?


is it a hardcoded password?


Enforcing dangles should be possible in the next version of ESLint (see: https://github.com/eslint/eslint/pull/1946).


شغل عالي يا معلم


inicio del comentario


It's ephemeral, but I'm pretty sure it's R/W. I can play around this weekend and see if it's possible?

-Samuel E. Giddins

On Mar 6, 2015, at 6:32 AM, Orta <notifications@github.com> wrote:

Yep

it'd be great to have that work! ( I thought the FS was read only? )
—
Reply to this email directly or view it on GitHub.


## This is a markdown file


d'oh!  good catch @tamarazuk copy-pasta


there is already test/ats/swfs/* entry in the root .gitignore


Nit, but shouldn't this be:
```
if (!(400 <= e.statusCode && e.statusCode < 500) || e.statusCode === 401) {
  throw e;
}

await this.deleteTaskFromQueue(task);
```

We shouldn't get responses that aren't 4xx or 5xx, but it's the internet you never know what you'll get :)
Also the control flow is a bit simpler, as non-4xx/401 errors causes throw, so no need to check status code again.

Remark: I don't know what you think the semantics of `this.log('claim task error'` is, but a 4xx error from the queue based on advice from azure table storage is a condition to be expected occasionally. Meaning it's not a bug, it will happen, but if it happens all the time we might have a bug (only might, there are valid cases where the azure queue is full of invalid advice).
So you might want to makes it very clear from any log message that this is not a bug.


why not
while(p!=NULL)
-> you wouldn't need the last nr++


both. You can use them but like written it yields to strange results. The look-behind-assertion even matches if the previous string was already matched. That is confusing for the user.
I'll make it more concrete^^
(Just remove "itself")


Done


Hm, why do we have `AST.Group`s?


It will be enforced on compiling your `(*libctFactory) Create`


@ipgray Can't do this one, need the institution


Looks like you dropped the `case LUA_TNIL:` which means it'll always throw `"array of strings expected"`?


Correction, we can parse multiparameter funtions, but not applications of them. The code generator is incorrectly implemented for this. The case I used while spotting this is:

b x y, where b is a function and x and y are arguments. This is translated to

(EApp (EApp (EVar "b") (EVar "x")) (EVar "y"))

I thought it would be translated to

(EApp (EVar "b") (EApp (EVar "x") (EVar "y")))

Is the first case the intended behaviour?


Hm. Very true, that should probably just return the inner expression.


Yes, but the grunt user has read-only access.


@ipgray Can't do this one, need S. Diaz institution


@frwickst this ``userId`` breaks the tests. Could you please correct this one. Thanks in advance.


I can see how this confusion happened.
I wasn't clear with the wording of "mark it if your working on it".

I meant attach your name to something your working on, but only check it when your done.


Possible, but that works nearly equally to the regex. If we use a simpler regex then we have higher memory consumption since the regex would match more times^^
It's either highly unlikely to use `\` itself when using the escaped function^^


Right, those entries were commuted accidentally.


Besides what I said above, good job.


Agregar mas etiquetas


Serwis społecznościowy dla amatorów gier komputerowych


model里面的方法，返回数据不要用json格式数据。请返回数组或者对象，然后在controller中使用model中的方法的时候，再进行json_encode操作。

另外，最好不要直接使用json_encode，如果要使用的话，也不能只回文本信息。需要回的基本信息应该包含状态码，文本信息，具体数据。ThinkPHP框架中自带了一个方法，可以直接使用，具体参照文档：
http://document.thinkphp.cn/manual_3_2.html#ajax_return


niewykonanie


What's wrong with `.shipped`?


Ohh sorry I didn't see that string double quotation should be added to the
message for file names for query message.Can u add ur change again?

But for response it is not needed right???

e.g., Suppose we are searching for string *baby*. So it will return, *0114
SEROK 3 129.82.128.1 2301 baby_go_home.mp3 baby_come_back.mp3 baby.mpe*

On Fri, Mar 6, 2015 at 9:44 PM, Supun Gothama <notifications@github.com>
wrote:

> Why did u change this? This is was correct.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/supungs/DistProject/commit/900f3d041765a255df642cbb5f6a37e61abb2154#commitcomment-10074791>
> .
>

--
Thanks & Regards,

Fathima Fahima
Undergraduate,
Department of Computer Science & Engineering,
University of Moratuwa.


wydaleniem


86046200_CondenserToZone (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


Done


oh, you wrote your own `rawgeti` function.


some tests failed:
failure: http://build.servo.org/builders/linux1/builds/946
failure: http://build.servo.org/builders/linux2/builds/944


Dear Adam,

You did a really good example on the auto complete from google, but when i try to copy your code to another project to use it it retrieves the data but doesn't view it on the table. Do you have a solution for that ??

Regards,


It is valid, and I could reproduce it 100% of the tries.

The only reason I did that, is that the **_surefire_** thread must block.

Any two threads(new thread 1 and new thread 2 or new thread 1 and surefire thread) fight for class loading, they initialize concurrently, yet need or want each other.
In OperatorImpl's static initializer it wanted to access all Ops, but in the mean time the 'static initializer' of Ops needed OperatorImpl to be loaded.

Does this description clear things up?


that'd be lovely


Projekt za zakończony uznaje się, jeśli:
a) zleceniodawca stwierdzi, iż jego obecna wersja spełnia jego (zleceniodawcy) oczekiwania i nie potrzebuje dalszej rozbudowy (projekt);
b) wszystkie założenia zostaną spełnione w terminie do 17 czrwca 2015 roku.


Why is `degree_names` an array and the expected return value is `[degree_names]`, which then expands to `[['bogus', 'test']]`


What does this button do?


I guess this depends on your other PR?


Nikt nie zgłaszał sprzeciwu, lub wszyscy go wybrali. Nie mogę się tam pchać tak na siłę.


saw approval from jdm
at https://github.com/JIoJIaJIu/servo/commit/bf26d61d5e63dd38723321542de9ee4d758056c6


merging JIoJIaJIu/servo/trace = bf26d61d into auto


JIoJIaJIu/servo/trace = bf26d61d merged ok, testing candidate = eda9c0c9


In which case the `switch` isn't needed, replace with:

     luaL_argcheck(L, lua_type(L, -1) == LUA_TSTRING, index, "array of strings expected");


Cannot post on internet yet, already have a file for embautista.github.io


Doesn't this get inherited from the base class?


I believe this is the same as `derive(Default)`, right?


30414/Dolineanu A. Vlad Ionut/Assignment 1 - extra/main.c

newCountry is such a confusing name here!


Celem projektu jest stworzenie strony internetowej, skupiającej wokół siebie społecznośc amatorów gier komputerowych.


@betrcode
FYI, the patch is still under review against JJB master:
https://review.openstack.org/#/c/150327/

Looks like the recent feedback requires a large number of changes...I will try to address those changes tonight or this weekend.


Personally I wouldn't do this.

I would force the user to create an environment specific deploy file from a deploy.example.rb. :)


Another perfect candidate for `[ci skip]`, I guess. :-)


True, but I figured that no fill/stroke is perfectly representable internally by `alpha=0`. There are bunch of other uses of Nothing in Compose and Gadfly that I need to modernize though.


89639540-AirflowNetwork-Crash (lgu1234) - i386-Windows-7-VisualStudio-12: OK (1646 of 1646 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)


Thanks for the clarification. I also tested it, and it works.


tnks!

when I have time, I will give a look!

sure!

2015-03-06 16:08 GMT+01:00 Kevin <notifications@github.com>:

> Hi, we had to move from GitHub. You can now register here :
> http://tyron.ovh to access the repo and work on it !! If you have any
> suggestions feel free to let me know ! [image: :smiley:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sniperal44/Server/commit/2c032825f0bb209b50d48871dc8189a87e06789a#commitcomment-10073346>
> .
>


Changes look good in general, I just see a missing word in the description of **featureName** (The name can **be**...), and probably we can remove the double quotes from **installFeature**


Hrm, the configuration files don't conceptually belong into the `mason_packages` folder, since it's managed by mason itself. For a rebuild, you should run `make clean`, which also deletes the gypi files, which will then trigger mason to redownload the files. To make this easier, we could instead also introduce a `make clean-config` or `make confclean` to just remove all config files.


Not sure if that's on purpose or forgotten?


89639540-AirflowNetwork-Crash (lgu1234) - Win64-Windows-7-VisualStudio-12: OK (1640 of 1646 tests passed)

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


which version of ocaml you have? I think they added |> in the standard library
only since 4.01


Is this because you and @englercj decided not to use brfs anymore? Personally I like the separation, but yes there was a bug here. I was about the make a pull request for the fix. Which was simply this:

```js
var AbstractFilter = require('./AbstractFilter');
var math =  require('../../../math');
var fs = require('fs');
```


I'd move this to MutableClosure and store it in a map (DeclarationDescriptor -> int) to leave EnclosedValueDescriptor as immutable, otherwise I can create the latter and forget about this parameter


30414/Dolineanu A. Vlad Ionut/Assignment 1 - extra/battleFunctions.c

if you're actually passing the file as a parameter, why not print
```
 if(nr==0)
        fprintf(g,"No country ");
    fprintf(g, "could have defeated all the sentinels\n");
    if(nr==0)
        fprintf(g, "%s could have brought down the first %d sentinels and would have chipped off %d life points from sentinel %d\n", bestName, turrets, chipped, turrets+1);
```
this part here?
This way, you can safely eliminate 3 parameters: ```turrets, chipped and nr```, because you no longer need to return them


It's quite alright that you have more parameters, it's normal. I like the new split into more functions, it's a lot cleaner than before


May need to toggle the sound upon the minimized state changes too.

As win_wndproc did this.


thanks for the info! there are too many options to keep track of!


@ipgray I need help on this one, I don't really understand it


Less code is good! Thanks for using my suggestion.


(y)


Unix首字母大写


You are right, a sleep is needed [here](https://github.com/mbraceproject/MBrace.Azure/blob/0bacba3d475d258674b1cff88d8c9d2f0b6d79c5/src/MBrace.Azure.Runtime/Worker.fs#L119) and [here](https://github.com/mbraceproject/MBrace.Azure/blob/0bacba3d475d258674b1cff88d8c9d2f0b6d79c5/src/MBrace.Azure.Runtime/Worker.fs#L156)


yup, too late, fortunately the travis queue isn't getting too much action today ;)


No need. Minimising triggers the lost focus event


Don't pass in degree names; Your test should say "degree names come from the repository if none are given"


Also Steph's diff.


Change to expect instead of allow


[didnt work :(](https://travis-ci.org/est31/minetest/builds/53359256)


Change to say 'when no degree is given and none is in the repository'


I get this:
450	06.03.2015 17:43:23	WARNING	Download failed: Dexter.-Stagione 05.-Episodio 01.Colpa.Mia.1080p.Bluray.Ac3.ITA.ENG.part01.rev | global name 're' is not defined


right


1: M111 unrecognized component name: general


you can fix that my using +> instead of |>, it's available from my common.ml
and there should be an open Common in this file.


I agree with @poutsma that a single cacheControl is a bit too generic. However `cacheContrlolMaxAge(int)`, `cacheControlNoCache()`, and `cacheControlPrivate()` goes in the opposite direction and limits the options too much unnecessarily. As soon as you need to add some directive (public, private, must-revalidate, proxy-revalidate), which can be combined with max-age, you're forced out of the cacheControlXxx convenience methods.

I think the general split should be along the lines of the mutually exclusive options, i.e. maxAge vs noCache with the ability to still add directives to maxAge:

    B cacheControlMaxAge(int seconds, String... directives);

    B cacheControlNoCache();


Keine Ahnung,wie das passiert ist. Ich hab hier einfach nur irgendwas gemacht.
Also, in der py-Datei, die ich dir in der Mail geschickt habe (übrigens die einzige komplette Version - das hier hab ich nicht weiter ergänzt) ist die Klammer noch drin. Ich hatte das eigentlich kopiert und hier eingefügt. Warum das jetzt anders als im Original ist, weiß ich selbst nicht.


Not to be done


@diffsky thanks for letting me know!


looks good.


Fixes Issue #10


this is not noteisher than the hint IMO so I'd rather keep it consistent


listen翻译成监听 是不是好一点


either.fold(throw _, _)


i dont mind either way :)  I prefer consolas, but it's all subjective ;)


@acacheung Let me know what you think about the formatting. Same for the Git Pro Book on line 46


Wenn du über patch-1 reingehst, ist die Klammer auch da


target instead of taget


Instead of the message with a single required field, you can just use that field:
```
repeated Callable secondary_constructor = 14;
```


what about Comic Papyrus?  http://laughingsquid.com/comic-papyrus-a-typeface-combo-of-the-two-loathed-fonts-papyrus-and-comic-sans/


correct me if I am wrong, but fu documentation should go here. The other text should be commented out (;;) if in-line.


@richardkchapman Please merge and close


maybe just `split`?


"十点"容易误解成时间


`item_matches` view will only return 404 only when `item_id` argument is not an integer. So this test will fail because 200 is returned. And here's the data from this response:

```
{
  "matches": {
    "current": null,
    "superseded": []
  }
}
```

http://ci.musicbrainz.org/job/geordi/28/testReport/geordi.api.views_test/APIViewsTestCase/test_item_matches/


最后一句有点费解


Fixed in 10eb93c - "Use WeakMap<Client, Front> rather than WeakMap<Target, Front>"


Hi otavio,

So, for PACKAGE_ARCH, that would be:
PACKAGE_ARCH_mx6 = "${MACHINE_SOCARCH}"

right?


Ha ha. Not much substance, just a whole lot of depth. :stuck_out_tongue_winking_eye:


It's a bit problematic to continue from here with message converters. A ResponseBodyInterceptor has the ability to change the body from null to some default value. It may be doing that for good reasons that do not apply to this use case. I think it would be more in line with the spirit of the change to do `getBody` in order to flush the headers and return immediately.


Yes I think so? The first expression is `(b x) y` while the second is `b (x
y)`.

2015-03-06 17:34 GMT+01:00 SoamJaws <notifications@github.com>:

> Correction, we can parse multiparameter funtions, but not applications of
> them. The code generator is incorrectly implemented for this. The case I
> used while spotting this is:
>
> b x y, where b is a function and x and y are arguments. This is translated
> to
>
> (EApp (EApp (EVar "b") (EVar "x")) (EVar "y"))
>
> I thought it would be translated to
>
> (EApp (EVar "b") (EApp (EVar "x") (EVar "y")))
>
> Is the first case the intended behaviour?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DavidLindbom/DATX02-15-29/commit/d6669ce90787d022f0ac9c1476fa3f2aa183c390#commitcomment-10075164>
> .
>


```
return computeSecondaryConstructors() + getUnsubstitutedPrimaryConstructor().singletonOrEmptyList()
```


so I mean `split` for just sane splitting and `unescaped_split` for splitting by sequence while the occurrence must not be escaped.


Unnecessary fat arrow


Line contains inconsistent indentation


它的URL API做这件事显得非常自然？


尽管如此，并不是每个人。。。


@ipgray I need their institution for this contribution


Is this the same python used to run Plone tests? We definitely do not want a default encoding there, because then we won't catch unicode errors in our test runs.


Like it.


In fact I believe we can reuse `member` field (tag 11) to also store secondary constructors


Yes, I missed that.  I'll push an update shortly.


shots fired


extra setLevel call


1: M111 unrecognized component name: general


以前，当用户开始构建仪表板，Graphite。。。不停地在。。。想象一下，你有一个。。。。


Alright. Is there a need for uncurried function calls in the TypeChecker? If not, I'll just make a local solution for this in the code generator. Or, if you have a better idea than to uncurry, let me know!


directory.listFiles(FileFilterUtils.fileFileFilter.asInstanceOf[FileFilter])


this doesn't need the string formatting, since line is already a string, right?


I'm going to create _RGWStatAccount_ObjStore_SWIFT::execute()_ and move the call there.


@glasser, much thanks for this! Why does the client code throw warnings about unsupported options? I thought the idea of Meteor is to have unified client/server calls. It could ignore or just log a warning.


I tidied this function up a bit in later commits. (e.g. d63b81fcd5)

It now looks like this:
https://github.com/joewalker/gecko-dev/blob/10eb93cb9e019ecf7127c3be9e402d67419dc5bc/toolkit/devtools/server/actors/gcli.js#L213

It no longer has the error cases.


See comment above (https://github.com/joewalker/gecko-dev/commit/19178eb762872d6dafa203971a531583a0faccc5#commitcomment-10075688)


I already fixed this in 0.2.3 version, but forgot to push.


“由于carbon可以毫秒级的速度写文件，所以还来得及。” 可以以


directory.listFiles().filter(!_.isHidden)


thing is we want to support arbitrary regex matching because we'll put some regexes into a language definition file. Thinking about this another issue arises... we probably want to support escaping by other characters than \\.


I think it would be cleaner to just add a new ```purge_on_error``` parameter to ```invalidate_cache``` instead of adding a member variable to ```ImageCtx```.


!(a & b & c) is not the same as (!a & !b & !c)

http://en.m.wikipedia.org/wiki/De_Morgan%27s_laws

On Mar 6, 2015, at 8:32 AM, Steven Petryk <notifications@github.com> wrote:

Could this just be where(public: true).not(latitude: nil, longitude: nil,
geofence_radius: nil)?

—
Reply to this email directly or view it on GitHub
<https://github.com/partygoer/partygoer-api/commit/7c4261f05007e5c275f3e35b317487c6958f4e7b#commitcomment-10071379>
.


and making it a generator shouldn't be hard


why'd u add a space there?


How about keep going and detect all files that have problems?


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (637 of 637 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-4.19%25-green.svg)


Looks pretty good, much cleaner now! :+1:


Well... what shall I do. the exception is *way* cleaner


No, I think was wrong - there is a 50ms timeout here which is adequate

https://github.com/mbraceproject/MBrace.Azure/blob/8d89480c79e12ae0e8d05677e8fec49859c1b8c8/src/MBrace.Azure.Runtime.Common/Resources/Queues.fs#L88

I don't think this code should have any sleeps? That would make job-dequeuing slower than it needs to be,


Mostly while I'm fixing tests it's a note to self right now. I'm thinking I'll probably remove them before we push to central, and see if anyone complains, but I'm not 100% sure.


Seems my commit was reverted indeed


I think it must be the victim of a mistaken rebase, yes!


ok


Oh right, I'll get on that first chance I get (probably tomorrow)


@jleclanche I'm not sure why do you ask me about this? I don't have the answer.


is this a cleaner

{% macro share(title, hide_header=True, twitter_options={}) %}

Share this page


space before {


space around { }


Just to give some example; says that a planet has 2 VertexHeightMap (like Vall). if you wanted to disable the 1st VertexHeightMap, and added another one, you would write something like this:

```
PQSMod_VertexHeightMap
{
			modEnabled = False
}
PQSMod_VertexHeightMap
{
}
PQSMod_VertexHeightMap
{
			modEnabled = True
			heightMap = GameData/Foo/Bar.png
			heightMapDeformity = 1500
			heightMapOffset = 0
			scaleDeformityByRadius = False
			order = 5
}
```


This is only enough to start playing a simple duty cycle from Sound 1 (no envelope, no sweep, ignores duration). Of course the code to generate the samples needs to be written still.


Fixed in bfe7779 → Firefox uses " rather than '


Ok, thanks for letting me know and also a BIG thanks for your hard work!
Den 6 mar 2015 17:44 skrev "wayne" <notifications@github.com>:

> @betrcode <https://github.com/betrcode>
> FYI, the patch is still under review against JJB master:
> https://review.openstack.org/#/c/150327/
>
> Looks like the recent feedback requires a large number of changes...I will
> try to address those changes tonight or this weekend.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/puppetlabs/jenkins-job-builder/commit/427bccfe5efdaa14e3e6eff93ae0a89de62fb630#commitcomment-10075331>
> .
>


Sim, de acordo com o enunciado só ganha pontos quem acerta vencedor, o placar vencedor ou ambos

Nesse caso vc utiliza o "public Time getGanhador()"

Certo?


Nevermind, I don't think this problem should be tackled until the "EVal" expression is integrated!


Moim zdaniem dostawianie LIB_DIR warto uzależnić od tego czy nazwa pliku zaczyna się od "/" czy nie.
Ktoś może przecież chcieć mieć plik opisujący dodatkowe uprawnienia w zupełnie innej lokalizacji niż lms.


That would work too, but you'd always keep two strings around in a Text instance which shouldn't be needed. Resetting `@decoded` would basically be the same as a boolean.


Комментарии лучше смотреть в файле, а не тут :)


I will probably move the general tips to another page at a later date, maybe when we have some more! thanks for writing this up.


"WTF" Seria no sentido de que não faz sentido ter um "getter" aqui, pois os atributos apenas serão utilizado dentro dessa classe?


This release only support ARM devices, preview version can support x86 and MIPS.


STOP TELLING ME ABOUT DE MORGANS LAWS YOU DE MORGAN EVANGELIST


It took me a minute to understand this.  I'm not sure identity is the right name.  Maybe `getValue`.  I don't know.  The confusing thing about identity is it makes it feel like a noop.  This is ok, but it confused me now and it probably will in the future.  What do you think?


in  socialdistribution/urls.py
Adding line 15 causes error on django


In the worker 99% of the time the `message, state` tuple will match this case `None, Running(config, _) `
This case has the 2 if branches. In the second case yeah the queue 50ms timeous is fine. The hot loop case is then currentJobs >= maxJobs, thats where a sleep is needed.

One thing that might be confusing is that the `Idle` state means that the MBrace.Service is Stopped. Not that the worker is waiting for jobs or sth.


Check once everyone....whether this code is ok....


Here is the dependency... currently it is hardcoded. Unfortunately it ends up with an exception:
https://gist.github.com/slaskawi/e20fa86763bd24995a01 line: 61


I trigger the build using
```
./build.sh clean install -pl server/integration/infinispan,server/integration/build -DskipTests && ./build.sh clean verify -pl server/integration/testsuite -Dit.test=CustomCacheStoreIT
```


Thanks for the explanation, now it's all clear to me :)

FWIW, I've just stumbled upon this question: http://stackoverflow.com/questions/6157497/node-js-printing-to-console-without-a-trailing-newline. The answers are worth reading through :)


Capture the problem in the inner scope to give more detail info.


Why? It's just a sample


The original constructs a Mutex for every Text node and locks it for every text access. I figured you'd prefer a lock-free option. You can clear @raw once decoded, of course.


I actually only tested it on Linux, but I'm glad to hear it works on Windows too :)


I think it'll work without it. The dynamic package arch class should do the right thing.


and… this hack doesn't work under Plone 2.7


I figured it out, in django 1.8 get_fields_with_model does load the app cache. In django 1.7 though it doesn't. I'm hoping once we get to django 1.8 we'll be able to do some cool thing like make LinkField virtual and bypass the whole issue.


86046200_CondenserToZone (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1106 of 1106 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-69.04%25-green.svg)


I feel like you can share more between Single and Multi layers.  May be make them return a seq or vector of the same type.  Single layer just returns a single instance of that.


Not really sure about these...


Beqare of q!


86046200_CondenserToZone (Nigusse) - i386-Windows-7-VisualStudio-12: OK (1604 of 1604 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)


all tests pass:
success: http://build.servo.org/builders/linux1/builds/947
success: http://build.servo.org/builders/linux2/builds/945
success: http://build.servo.org/builders/android/builds/983
success: http://build.servo.org/builders/mac1/builds/1162
success: http://build.servo.org/builders/mac2/builds/1163
success: http://build.servo.org/builders/gonk/builds/133


@nicosang  - Why the deviation from what the rest of the project uses for logging?  What is the difference and reason ?


This should be default!

![comic](https://cloud.githubusercontent.com/assets/115237/6530064/088267da-c42e-11e4-9285-5be72c3ef6db.png)


Life is about choices :)


if the file is too big it's gonna complain about java heap


@KristenEgermeier can you note the level of sponsorship?
@mikelmaron and all... are we going to note level of support on the website?


Figure out the find target project exception and catch that instead of catching the negative of IOException.


what is this?


OK


Hey Garry, sorry for the delay in addressing your PR. We appreciate your effort here, but when we created the Emojione library we decided to not support the legacy encoding formats since they are rarely used anyways and maintaining support for them could prove to be too time consuming and problematic for us.


fast-forwarding master to auto = eda9c0c9


А я подумал, что вариант с работающим методом лучше, чем с исключением. Или
контроль за аргументами надо возлагать на тех, кто будет юзать библиотеку?

2015-03-06 19:10 GMT+03:00 AnzhelikaKravchuk <notifications@github.com>:

> Первая идея с ArgumentException мне нравится больше, чем возврат -1.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MatthewKoreysha/EPAM.Homework/commit/557a330c54877387644aca4fd514a8ca6cff8a92#commitcomment-10074692>
> .
>


The “avaible” (mistaken “available”) here and on the line 41.


Resolved, thanks!


What version of the .psm1 did you base this off of? I changed Print-MSDeployCommandString to Print-CommandString back on 2/1 https://github.com/sayedihashimi/publish-module/commit/c34802d819022f489ccafaf66cf6ecd04cb4de16.


My bad, guess I assumed.


I didn't see very good results from this magnetometer and haven't been using it. I would recommend the LSM303C accel/magnetometer instead.

Of course, the variable heading could be a result of improper circuit design. Where did you get the FXO8700CQ sensor board? And which microcontroller are you using?


use with Logging


The block above should be split so `vagrant ssh` and `cd /vagrant` are in their own block from `npm start`, because the *or* here only includes the startup with token.


saw approval from jdm
at https://github.com/cyndis/servo/commit/92a4bba7854ebc25598174df3300a3a27101ab28


cyndis/servo/check-font-result = 92a4bba7 merged ok, testing candidate = 4bd5aed5


merging cyndis/servo/check-font-result = 92a4bba7 into auto


Please use the existing pattern of

```
'my message' | Write-Verbose
```

This makes it easier to scan through the functions to find statements that are missing ```Out-Null```


LGTM


This is a test comment


Please do not uses aliases, expand ```cp``` to ```Copy-Item```. It's also better to specify parameter names instead of relying on positional parameters.


Please notice this line.


This commit is to cover the feature.


Provide a meaningful name for ```$m```


W sumie mozna by bylo
Poprostu przesiadam sie na nowa wersje i przestalo mi to dzialac spojrzalem w kod Diry a tam bylo z prefixem LIB_DIR


Are you good with SSHing into the servers to upgrade?


I believe some of these tags/styles deserved at least a macro at the top of the file... that could be extracted out later when we decide to use them for other things.


True.  I like that you matched what's already used elsewhere; however, the HTML5 file system API's 'size' parameter is also, really an estimated size.  I'll leave this to @brianleroux to decide whether to go with the SQLite name 'estimatedSize' or the file system API name 'size'.  I'm discussing this because I think calling it just 'size' is misleading because it sounds like a statement of fact, not a suggestion (which it is).  Good work, @tricki.


It would be better to have the same returned type between Single and Multi layers.


Already doing that on hub.18f.us; it's already installed on 18f.gsa.gov.


I want to make this backwards compatible with 3.0.  Not sure what best practice is for doing this.  My first inclination is something like:
```
related = info.related if hasattr(info, 'related') else info.related_model
```
Thoughts?


Please do not use the ```return``` statement. It doesn't do what you expect. In PowerShell any values that are not captured by a variable or ```Out-Null``` will be returned to the pipeline. So in this case if the if block is executed you may end up returning both ```$value``` and ```$m.value```. I'd change this to be more like the following.

```
if($value){
    $value
}
else{
    $m.value
}
```


with Logging


Now you're pushing it a little bit.


either.fold(throw _, x => x)


"ALREAY" ?
May be "ALREADY" ?


We moved watch from the build task to the default task on the Esri branch, https://github.com/Esri/esri-leaflet-renderers/pull/35/files.  The watch task is not meant to complete, it's purpose is to watch your files and rebuild the distribution files whenever you edit a file.  If you don't want this behavior and you are using the Esri branch you can now just run `grunt build`.


tabs vs spaces


This was copy-pasted from the code you wrote for livesync ;)


Same mistake as 3 lines upper


True. Didn't think of that. :+1:


poTAto, poTAHto


fef


debug leftover


True. I'll catch `APIException` instead


According to http://docs.scala-lang.org/style/scaladoc.html that space should be there.
I think it looks a bit weird too, but I looked at the style guide.


make sure this is not there in prod


Not according to PEP8, AFAIK ;)


Could you include a rationale in this commit message?  Something like:

"These utilities are not actually specific to the `ostree` commandline".


Ben, I'm not sure you did the right thing here. Our `master-os2` branch is based on the v4.3 and at that point there was **no** README.md in the root, check yourself: https://github.com/bitwiseworks/eclipse.platform.swt/tree/b05ed3b2410bf9880ea77e7dcf0101adc35a621a. You should not break history in this way as this may bring really big problems with future merges. In this case you should just have renamed README.md to README-OS2.md. Please correct this.


Yes, good idea.


Hey guys, some people call this type of import:

```python
from glfw import *
from OpenGL.GL import *
```

as "bad practice". I understand that in some cases it is necessary (from inside a __init__, from example ). But in some cases, it just creates difficulties in code navigation. For more discussion (about it:)[http://stackoverflow.com/questions/2386714/why-is-import-bad]

For instance, could you give some tips for code navigation with the Sublime Text? :)


> is this cleaner ?

I re-wrote it that way, and I agree it's cleaner for the macro, but then looked at the implementation and it seems counter-intuitive to me from the implementation standpoint.

Here's what the implementation would look like with `{{ ''u-visually-hidden' if hide_header }}`:

```
share(post.title, true, {
```

So that means include the share buttons, reference the post title, and DON'T show the heading (true = hide_header). If I was just looking at that without looking up the macro I would guess the true would be for DO show the heading. So it seemed clearer to leave it as show_header (also means the existing implementations don't need to be updated).

However, I just realized we are doing this for the showing/hiding headings in the tags too, right? This behavior should probably be consistent between the two. With the implementation in mind any preference on the two approaches? cc @jimmynotjim I think you originally flagged this for the tags?

> Also, it should be 'False or True'.

Mm, that's right, fixed in https://github.com/cfpb/cfgov-refresh/commit/be6c03425803af4d3400f5f91a34d4cbcf266011. What's the difference between the case in Python, it seems like lowercase works the same? Or would that mean the [false in this](https://github.com/cfpb/cfgov-refresh/blob/gh-pages/events/_event.html#L45) is wrong?


use `$c[$pri] = sprintf('%02x', round(...))` instead?


It could be more  idiomatic to say "getPodInfraContainer finds an ..."


Good point.


I see what you did there


r+


ditto


Also removed unused variable ouah in sendX()


What is this used for? I don't see `skin_options` elsewhere, can this be removed? Suppressing errors with `@` is also frowned upon.


What is "Execute-PowershellCommandString"?


86046200_CondenserToZone (Nigusse) - x86_64-MacOS-10.9-clang: Tests Failed

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


Why suppress errors? If `rrdtool_colors` does not have the member it is a bug? What about making a method of it for the sake of abstraction an introduce a Skin interface?


Attention Brian. I have updated index page to look for your page when logging in as that user. Will delete old page of mine on next commit.


Can tmpFile be val?  space after ':'


This has already been added previous - need to sync up your fork with the latest.


@judgej can you cover this with a test? Then I'll gladly merge


i guess you could also just change the security zone settings in the image, right?


@gwicke: Do you know if we can simplify res.end(JSON.stringify({error: "q (query) post parameter is missing!"})) to res.json({error: "q (query) post parameter is missing!"})


android-channe


The node variables are not openfisca-france variables, they are the sum of the leaves.


It is based on latest version from master, I added another generate Print-CommandString command, so I renamed the existing one to Print-MSDeployCommandString because it is indeed only for MSDeploy command.


I think it's more common to see a single line when error is the only return type.


What's up with this sleep? Minimally the 5 shouldn't be hard coded but instead a global setting in ```$global:AspNetPublishSettings```


Here's the line:
```html
<%= link_to "Delete", @note_path, method: :delete %>
```

Here are some things to consider to help figure this route out:
- `_path` helpers are **methods** that return `String`s, they're not instance variables. You can try this in a view template as instance variables don't allow you to execute them with a `()`-suffix, e.g. `@instance_var()` won't work. `something_path()` will work. Methods don't begin with `@`.
- Look at the output from `rake routes` for the relevant delete route. Look for required `:dynamic_segments` such as `:id` and `:something_id`. These dynamic segments need to be filled in from data passed into the `something_path` helper method, e.g. `something_path(something)`
- `method: :delete` option is needed and this looks correct above.
- Experiment in the rails console, with code like: `app.something_path(Something.first)`


Please make these strings translatable.


use tmpDir.foreach


Why we are changing this?


Please update the existing ```Execute-CommandString```


this set is empty, why do you need to initialize from empty cuncurrent hashmap.


Sorry Status weiterhin Setup


space after operator >=


Is there a valid default port?, else this code is fine.


```{``` should be on the same line as the code.


dont like this being associated with deployments


I still don't understand this. Why do we need to retrieve this data here if we are not going to use it? Yes we are now saving lines but it's the same concept behind.

If a function needs to use the prefix it should then get it and if not set then get set the default to config and return that value.


We shouldn't have two different methods instead we should adapt the existing function.


Hey chi, this line does not exists in original SingleLayerShapeFileReader. Do you think there will be a problem adding it in? I think its really an issue that depends on file-system you are using, but either way it is something we should be doing regardless.


ahh, ok got it - will have to add this myself as I can't merge your changes


The logic following this "if" would be easier to follow if refactored into a function. The same for the other one.


mis-spelled, should be collapsible


You may be right. I'm basing what I said about ``sscanf()`` on what the man page about the possible values of ``errno`` which suggests on interruption is possible.

```
ERRORS
       EAGAIN The file descriptor underlying stream is marked nonblocking, and the read operation would block.

       EBADF  The file descriptor underlying stream is invalid, or not open for reading.

       EILSEQ Input byte sequence does not form a valid character.

       EINTR  The read operation was interrupted by a signal; see signal(7).

       EINVAL Not enough arguments; or format is NULL.

       ENOMEM Out of memory.

       ERANGE The result of an integer conversion would exceed the size that can be stored in the corresponding integer type.
```

Although perhaps ``EINTR`` only applies to ``fscanf``, ``vfscanf``?


use finally or something to ensure that ops.close is called.


OK. I'll update it.


not sure the widget bit is necessary in the name, maybe just osc-key-values


ah, yep, you're right


or osc-key-value-editor


I can't read the above links.


Remove space like managed(new Zip...


i'd favor having these two lines before the comment which applies to the line after this...


Is there a better way to set this up so things will still work even if mica.web and find_attitude.web are not installed?  Or is that just too much checking code?


lol


Agreed! will do, I will collect errors and send these out to customer as one message.


added this manually - couldn't merge - thanks


This is awesome. Thank you!


Anyway, could I rejoin the org. I am trying to unf**k the history


Which I have done in c6ae423baa82eb6fefa55fcc0bf5dad37b9d35cb.


@snostorm Is there a reason this should be merged with `project.json`? I'd prefer the data for this be separated from the logic, and having a separate json file makes it easy to update that without touching actual code. Keeps things a little more atomic.


In a relationship like this, one side is `has_one`, and the other side is `belongs_to`, so this line in `Summary` would become:

```ruby
class Summary < ActiveRecord::Base
  belongs_to :post
```


I hope to seee more sharing and a smooth transition between Single and Multi layers.  I also hope Multi layers do not use much more working memory than each individual layer.


I was not sure about this. I've never had to manage a project directly depending on another one, but it seems like a good idea to want to stay backwards compatible.

Codewise, this seems reasonable (with a comment explaining it). How will you decide when it's time to drop support for 3.0 ?


shoot sorry about that! I actually needed a working version as we are using this for a project.


For future reference: https://twitter.com/holman/status/557275998341709824


Needs refactoring, avoiding to duplicate code from PMA_updateOneColumn().


Точно! Код клиент всегда знает набор исключительных ситуаций для используемого метода. И он может либо позаботиться о вводе корректных для метода значений, либо отлавливать и обрабатывать исключения. Ну а -1 может быть, например, результатом извлечения корня 3-ей степени из -1.
--
С уважением, Кравчук А.И.

> On 06 Mar 2015, at 20:28, MatthewKoreysha <notifications@github.com> wrote:
>
> А я подумал, что вариант с работающим методом лучше, чем с исключением. Или
> контроль за аргументами надо возлагать на тех, кто будет юзать библиотеку?
>
> 2015-03-06 19:10 GMT+03:00 AnzhelikaKravchuk <notifications@github.com>:
>
> > Первая идея с ArgumentException мне нравится больше, чем возврат -1.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/MatthewKoreysha/EPAM.Homework/commit/557a330c54877387644aca4fd514a8ca6cff8a92#commitcomment-10074692>
> > .
> >
> —
> Reply to this email directly or view it on GitHub <https://github.com/MatthewKoreysha/EPAM.Homework/commit/557a330c54877387644aca4fd514a8ca6cff8a92#commitcomment-10076071>.
>


well, @drkibitz showed the fix :)


Well done on getting the `form_for` right here. An alternative that I believe would have worked too is:

```html
<%= form_for @summary, url: topic_post_summary_path(@topic, @post) do |f| %>
```


Thanks for the suggestion! :-)


Actually no reason. It's a leftover from other try, and one of the things I learned, that using := when assigning to pairs is problematic. This branch will go away in the next PR.


Leave InvalidShapefileSet as a collection.


Il GrappaFSD, più che un LUG, è un gruppo di amici (2) che scrivono software open source (il cui codice però è impossibile da trovare in qualsiasi meandro del web). Siamo sicuri che debba essere incluso nella LugMap?


You should be checking if argument length **is** 0. With your current check, the only thing that can be done is "/bcast".


Блин, действительно) Я как-то не подумал о корнях нечётных степеней)

6 марта 2015 г., 20:54 пользователь AnzhelikaKravchuk <
notifications@github.com> написал:

> Точно! Код клиент всегда знает набор исключительных ситуаций для
> используемого метода. И он может либо позаботиться о вводе корректных для
> метода значений, либо отлавливать и обрабатывать исключения. Ну а -1 может
> быть, например, результатом извлечения корня 3-ей степени из -1.
> --
> С уважением, Кравчук А.И.
>
> > On 06 Mar 2015, at 20:28, MatthewKoreysha <notifications@github.com>
> wrote:
> >
> > А я подумал, что вариант с работающим методом лучше, чем с исключением.
> Или
> > контроль за аргументами надо возлагать на тех, кто будет юзать
> библиотеку?
> >
> > 2015-03-06 19:10 GMT+03:00 AnzhelikaKravchuk <notifications@github.com>:
>
> >
> > > Первая идея с ArgumentException мне нравится больше, чем возврат -1.
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> https://github.com/MatthewKoreysha/EPAM.Homework/commit/557a330c54877387644aca4fd514a8ca6cff8a92#commitcomment-10074692>
>
> > > .
> > >
> > —
> > Reply to this email directly or view it on GitHub <
> https://github.com/MatthewKoreysha/EPAM.Homework/commit/557a330c54877387644aca4fd514a8ca6cff8a92#commitcomment-10076071>.
>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MatthewKoreysha/EPAM.Homework/commit/557a330c54877387644aca4fd514a8ca6cff8a92#commitcomment-10076444>
> .
>


With the association altered as suggested below (by introducing `belongs_to`), line 24 could be simplified to:

```ruby
@summary = @post.summary
```


oh nice! sorry bout that.. peddle to the metal on a prototype here today!


Yes, but then err won't be declared in line 1136, I'll need to use :=, which will shadow podStatus. It won't be a problem in the next PR - I'll fix this there.


I don't want to run the command via cmd.exe, my commands are powershell commands, I tried to update the existing Execute-CommandString at first by adding a switch useCmd, it looks like below, do you think it is ok?

function Execute-CommandString{
    [cmdletbinding()]
    param(
        [Parameter(Mandatory=$true,Position=0,ValueFromPipeline=$true)]
        [string[]]$command,

        [switch]
        $ignoreExitCode

        [switch]
        $useCmd
    )
    process{
        foreach($cmdToExec in $command){
            'Executing command [{0}]' -f $cmdToExec | Write-Verbose

            if ($useCmd) {
                cmd.exe /D /C $cmdToExec

                if(-not $ignoreExitCode -and ($LASTEXITCODE -ne 0)){
                    $msg = ('The command [{0}] exited with code [{1}]' -f $cmdToExec, $LASTEXITCODE)
                    throw $msg
                }
            }
            else {
                try {
                    Invoke-Expression -Command $cmdToExec
                }
                catch {
                   if(-not $ignoreException) {
                       $msg = ('The command [{0}] exited with exception [{1}]' -f $cmdToExec, $_.ToString())
                        throw $msg
                    }
                }
            }
        }
    }
}


I think it does a disservice to users if they pass a potentially vital option to completely ignore it on the other platform.


option.toRight(InvalidShapefileSet(...))


Good use of the singular `resource` method here :+1:


Totally forget that that commit is all about nice lollipop-like styles


With the summary belonging to the post, this migration would become:

```ruby
add_column :summaries, :post_id, :integer
add_index :summaries, :post_id
```


This is awesome


The kestrel server need some time to start up, will move to global settings


Just curious why exported? Are we expecting to call it outside kubelet?


well it is an identity in that it just passes the value directly through.  http://underscorejs.org/#identity


But to no avail...


<!-- PMABOT:SOB -->
This commit is missing Signed-Off-By line to indicate that you agree with phpMyAdmin Developer's Certificate of Origin. Please check [contributing documentation](https://github.com/phpmyadmin/phpmyadmin/blob/master/CONTRIBUTING.md) for more information.


Perfect!
Thx man! :dancer:


Plus, it doesn't seem to be possible. click has an `.argument` decorator that flask-script doesn't seem to use.


To your first concern, the answer is that we probably would move away from using SIngleLayer approach in the near future, but the functionality changes are substantial and will require a small discussion and some refactoring, so this is meant really as an intermediate step: get this right then proceed on to the bigger problem.

As to the later question, Yes I have wondered about this too, but that would require more control for the user over the directory with the contents parsing those items and then feeding them to the readers one by one, which would be ok I suppose but breaks the original functionality of the reader and really becomes more of a parser. If that is a better solution than yeah let's do that? Anyone else has an opinion on this?


How about use the general Print-CommandString, so all MSDeploy command will format its own message pass in Print-CommandString?


Yes it would be better to have one consolidated function. Can you update ```ignoreExitCode`` to ```ignoreErrors``` so that it applies to both cases.

Change ```useCmd``` to ```useInvokeExpression``` and have ```cmd.exe``` be the default.


Thanks, I'm still a powershell newbie, will fix this.


Tab issue?


Yup :+1:


OK thanks. Have you contacted asp.net runtime team to see if there is a better way to do this? What happens if the user has a slow machine (or VM) and 5 seconds is not long enough?


How can we handle the timeout then?


Right, will do.


Wouldn't this change cause issues for users who already have their machine setup with gcode_flavor = "Mach3" and with the change, the gcode_flavor becomes unknown as it won't satisfy the if gcode_flavor == "Mach3/LinuxCNC".
I'd suggest checking for "Mach3" and replacing the gcode_flavor to "Mach3/LinuxCNC" to avoid a regression for previous users.


Here, the set needs to be the concurrent set.  So, Creating the empty concurrentset from the concurrentHashMap.  Below line shows, adding the kafka partitions to this empty set.


Skipping hidden files seems reasonable to me unless shape file has a scheme to hide some files that are required for system processing.


Possibly a QT cross platform issue with nested templates and widgetgroups.
The Mic widgetgroup is drawn underneath the deck, even if the xml says otherwise.
Comment out the deck template https://github.com/mixxxdj/mixxx/blob/master/res/skins/Shade/skin.xml#L145 and the Mic section becomes visible.
![2015-03-06 18_20_40](https://cloud.githubusercontent.com/assets/4525897/6530694/5a0ae9a6-c433-11e4-86c5-c6611c137944.gif)

```bash
user:~ useri$ qmake -v
QMake version 2.01a
Using Qt version 4.8.6 in /Library/Frameworks
user:~ user$
```


Mee tikath query executor ekata daamuda? file name eka witharak yawala. ethakota lesiy neda?


@bobylito better do `if req.setTimeout..`


What is the advantage of Logging?


@redox so the currznt state was that we added a timeout but no listener, which means: no callback or retry if timeout.

the nodejs client seems to require some love maintenance thought.


Please rework that file, so that it fits to the DE2i-150 board!


why not, only important that the escaping thing is documented inside the description^^


Add your copyright here also


On Fri, 06 Mar 2015 00:29:06 -0800 Christian Loos
<notifications@github.com> wrote:
> Shouldn't this compared against the $MaxAttachmentSize config instead
> of 5M?

Hm, interesting idea.  Unfortunately, from RT_Config.pm:

    C<$MaxAttachmentSize> sets the maximum size (in bytes) of
    attachments stored in the database.  This setting is irrelevant
    unless one of $TruncateLongAttachments or $DropLongAttachments
    (below) are set, B<OR> the database is stored in Oracle.  On Oracle,
    attachments larger than this can be fully stored, but will be
    truncated to this length when read.

So because it's documented to no do anything in most configurations, it
seems somewhat wrong to suddenly make its value important.
 - Alex


Nit: I think I'd still prefer to see this error reported on the impl (i.e., in coherence), but it's a minor point.


Actually had a discussion with @rjmac about this. The answer is that Try should be discouraged from being used in this context, and as such we should move towards using Either. In order to move towards this direction I think it would be best to continue as is.


add a comment like "disable xhr credentials flag in browsers" with a link to http browserify readme on this maybe? or the xhr spec.


RamereWimberly


That makes sense, thanks.


wondering what happen when parse is defined. this whole Parse part is not done "well"


Yeah fixed that on my branch, not the master though


This should probably be broken out into a separate file since it isn't related to apt_updates.


==> DIST SETTINGS:
  -> dist_name: Manjaro
  -> dist_release: 0.9.0-dev
  -> dist_codename: Bellatrix
  -> dist_version: 2015.03
==> ISO SETTINGS:
  -> iso_label: MJRO090D
  -> iso_name: manjaro
  -> iso_compression: xz
  -> iso_checksum: md5


The kerberos issue should be fixed now. I'll look into the libiconv logic.
On Mar 6, 2015 7:07 AM, "Daniel Peebles" <notifications@github.com> wrote:

> Why do the libiconv or null logic when libiconv itself already has
> fall-through logic on linux? Does the glibc dep not work?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/936fdc7b68335a32b6c29df774044d46d46ff2ca#commitcomment-10073302>
> .
>


add file header (similar to header I use for VHDL, but use Verilog comments of course ;)

Please give info what the heart beat ipcore is used for?


FYI some higher level feedback. I noticed that the docker script doesn't support the following.

- [ ] Replacements
- [ ] Exclude files

Not sure if you want to support these but it would be nice. It's not needed for the initial release though we can revisit that later.


Add your copyright


It sounds to me like you may have forgotten to connect the UItableview datsource and delegate on storyboard.

Would you mind sharing your project with me, so that I can quickly take a look?

Thanks,

-Adam

> On Mar 6, 2015, at 10:40 AM, madmood <notifications@github.com> wrote:
>
> Dear Adam,
>
> You did a really good example on the auto complete from google, but when i try to copy your code to another project to use it it retrieves the data but doesn't view it on the table. Do you have a solution for that ??
>
> Regards,
>
> —
> Reply to this email directly or view it on GitHub.
>


Unfortunately this fix also modifies all paths even when `config_path` is not provided.

I've moved this block after `config_path` is setup.


Have you tested this on a system where you explicitly removed gpg? Not sure if `Facter::Util::Resolution.exec` is good a catching errors, if not you may need to wrap this in a try/catch and return nil if gpg doesn't exist.


Err, ok. Ahem. Force-pushing.


Probably want this to be `if $::gpgversion and versioncmp()...`


oh, and that should be `<`, not `>`


It was a decision coming from my team dev lead.  To me, it means less typing.


This does not seem right!
Why one need to do this?


align the indentations please


:confetti_ball: :+1: :hamburger:

:hamster: :hammer:


all tests pass:
success: http://build.servo.org/builders/linux1/builds/948
success: http://build.servo.org/builders/linux2/builds/946
success: http://build.servo.org/builders/android/builds/984
success: http://build.servo.org/builders/mac1/builds/1163
success: http://build.servo.org/builders/mac2/builds/1164
success: http://build.servo.org/builders/gonk/builds/134


maybe it makes sense to add info that this file is for the de2i-150 demo


First draft. Yup yup and yup. Tests to follow as well.
On 6 Mar 2015 18:11, "Morgan Haskel" <notifications@github.com> wrote:

> oh, and that should be <, not >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tphoney/puppetlabs-apt/commit/414742699f7ef70b729bea1128c622f734a35004#commitcomment-10076693>
> .
>


The problem is what does it mean to make an escaped split with an escape character?
So how would you split this sequence:
`a\\\\\b`
In my opinion you have at least two options:
- `a; ; ; ; ;b`
- `a\\\\\b` (the last `\` is escaped, that's what's my function doing and I think applicable for this example)
- and maybe further combinations.

But I had weirder strings that were strangely splitted sometimes, that's why I've written CAUTION.


Maybe it had the wrong value set, but if you remove it then it will be never displayed unless that variable gets passed because of other feature.


wrong board name in brief


your copyright


Rework the makefile like done in commit 0093215b8fbd711b80eee13a8483970d795a01f7 [TASK] Rework Quartus project makefiles


fast-forwarding master to auto = 4bd5aed5


> Is this because you and @englercj decided not to use brfs anymore

No we are using brfs, didn't realize that their transform doesn't find fs properly unless it is declared alone. That is silly.


Query generated by your version:

`SELECT "parties".* FROM "parties" WHERE "parties"."public" = 't' AND ("parties"."latitude" IS NOT NULL) AND ("parties"."longitude" IS NOT NULL) AND ("parties"."geofence_radius" IS NOT NULL)`

Query generated by my version:

`SELECT "parties".* FROM "parties" WHERE "parties"."public" = 't' AND ("parties"."latitude" IS NOT NULL) AND ("parties"."longitude" IS NOT NULL) AND ("parties"."geofence_radius" IS NOT NULL)`

ActiveRecord doesn't treat my expression as !(a & b & c), it treats it as (!a & !b & !c).


Pretty sure it's fixed on Konrad's branch too


... bitch.


LINE 10 - indentation and Then step


- Rework that file like done in other qsf files
- Add file header


It's a default clusterId.


When `path.relative()` returns an empty string you end up with `/relative/to/absolute/paths`, so, using `path.join()` would be enough.


Yes, I plan to to keep both examples and move the simple example higher so readers see it first. Thanks for reviewing the changes.


Replace with common_steps


@rcrowder Thanks! added


I guess you will get warnings for every unused pin, right? Remove them if so (avoid warnings)


We are not only saving lines, but we are also consolidating logic, simplifying the code and since the config is the only   externally accessible property. Plugins can get all the information they need about the controllers through it.

Also since the input can be manipulated (which causes side effects) getting the information in the config here prevents developers from breaking other controllers.


I've published minor tweaks to your changes for passing CI, could you run your tests again?


82420068_GridViewMultiEdit (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2149 of 2182 tests passed)

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


Thanks Eliot.  I'll keep working on it.  I realize that  ```@note_path``` was not a good solution, but it was the only thing that yielded a result (i.e. it destroyed the participant) : )

I did look at the output from ```rake routes``` and what I saw for ```notes#destroy``` is ```participant_note```  and ```/participants/:participant_id/notes/:id(.:format)```

This was my first attempt:
 ```
participant_note_path(@participant)
```
followed by several iterations of the above that also did not work.

I'll continue to work on it and let you know what I come up with .  This route stuff is working me over.  Sorry for all the question about this....


saw approval from mbrubeck
at https://github.com/metajack/servo/commit/aedd157035778a8edc7e64a9ed2b24a78a62687d


merging metajack/servo/fix-post-build = aedd1570 into auto


metajack/servo/fix-post-build = aedd1570 merged ok, testing candidate = 23b65a05


To set IO standard for a bus, please check if you can set it also for a group..

       set_instance_assignment -name IO_STANDARD "3.3-V LVTTL" -to FS_DQ[*]


Am 06.03.2015 um 16:08 schrieb Kevin:
>
> Hi, we had to move from GitHub. You can now register here
> :http://tyron.ovh to access the repo and work on it !! If you have any
> suggestions feel free to let me know ! :smiley:
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lyrex/Server/commit/0d2fa26caf45992a58cd8c752d1edceeb8ff2079#commitcomment-10073330>.
>
Hey Mate,

for me your Repo does not work. I can't checkout or anything since it
says that the connection was refused.
Did you probably do some misconfiguration?

Kind regards


This is exactly the same as the one at line 120, isn't it?


зачем закрывающий тег на новую строку пареносить?
незачем.


This needs to be changed to include the options allowed.


Do not repeat the pkg name in general. This will look like rocket.RocketRuntime, it is better to have rocket.Runtime.


I confirm that this patch works. Are there any plans to commit this feature back into the official ISC distribution?


Column `product_attribute_shop.id_product` doesn't exist, causes `Db` exception. The old SQL was

`LEFT JOIN ps_product_attribute pa ON (
  p.id_product = pa.id_product
)
LEFT JOIN ps_product_attribute_shop product_attribute_shop ON (
  product_attribute_shop.id_product_attribute = pa.id_product_attribute
  AND
  product_attribute_shop.id_shop = 1
  AND
  product_attribute_shop.default_on = 1
)`


OK, so if the test doesn't exist on the other side, the avocado error message is relayed to the user. Now, we are supposed to print the error messages to stderr. I wonder if this is a side effect of fabric merging the streams.


- Convert absolute to relative paths
- The SLD_FILE assignment can be removed as far as i know


https://code.google.com/p/go-wiki/wiki/CodeReviewComments#Error_Strings

cannot get the version of rocket.


@karimnaaji `feature.id` is not something that we can expect all data sources to have -- it's present in our vector tiles, but is not a standard


probably describe the error case a little bit.


Hi,

Update your database.

https://github.com/PrestaShop/PrestaShop/blob/1.6/install-dev/upgrade/sql/1.6.1.0.sql

Regards


Optimistically, you could change your workflow instead of the code. :relaxed: The point is that images are images, and there's a directory for them. In a parallel matter, JSON texture atlases may be included in the image directory as a consequence of TexturePacker's default setting placing both outputs into a single directory. I suppose that could be improved as well.


`ssl:close/1` can block for a significant time in certain situations IIRC, might need to spawn a process to close it.


88621024-ElectricLoadCenterStorageBattery-Crash (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1648 of 1648 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)


Need to add a struct field to hold options so we can use it in PrintDefaults.


Remove signal tap before final integration


Same. I stopped Sickbeard, then used:

git reset --hard ff36290ba50a08d004b41cbcb8649645c324797a

then I restart Sickbeard and setup Plex again. Now it's working.


это и есть h1


sphinx-build errored here.
/home/travis/build/SpongePowered/SpongeDocs/source/workspace/text.rst:28: WARNING: Title underline too short.


Since this should always be the same, can we make this a constant?


download__btn — потнятнее, кажется


k8s people do not like make. I usually map[string]string{} in k8s codebase.


Please check which ports to you really use. Remove those which are unused. Also check if it is necessary to set a constant value e.g. to disable an unused device!


I do think so; the concat task reads this file from the build directory. https://github.com/melonjs/boilerplate/blob/02ef03662e7f5f4acc6813a3ef7553b00f903a3c/Gruntfile.js#L6

index.html as well: https://github.com/melonjs/boilerplate/blob/02ef03662e7f5f4acc6813a3ef7553b00f903a3c/index.html#L29

The `js/resources.js` path is no longer referenced anywhere except this change.


Do we really want a slice here or just a map[string]*ConfigVar?


This is a breaking change for all modules depending on `browserify-transform-tools`: instead of expecting the transform config from `opts`, they receive them in `opts.opts`.


Hm then we should probably do that with the other close calls that were already existing. :-)


It's ok to disable max-len in the test cases because there are so many long strings, but it's not ok to turn off everything.


rocket starts to run pod:


Feature Tags
@usat-mweb-fronts @usat-mweb


All looks good to me.


это section


if the error will be propagated outside the pkg, we need to add a pkg prefix. rocketRuntime: unexpected


I'm not sure I'd make the whole `Individual` an atom, but instead just the parts of the individual that you want to update. So you can have just `(atom {})` to indicate that the errors part of the individual will be updated, but the other parts (like the UUID) remain immutable.


poke @Grz3s
how do we apply the statement above to the other difficulty entries for each creature, without putting in all the entries.
Do you have any template for this?


Yes, but please don't ragequit again :)


Yes, to be discussed and worked through shortly.


You are not really navigating to all the sections. I would change the scenario step title 'Navigate to all the menu sections' and then create another scenario for random subsections.


зачем .price__table-cell ?
можно обойтись без них, привязываясь к td каскадом (выше — th, а для еще большей гранатии можно испольховать thead и tbody)


82420068_GridViewMultiEdit (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Not sure if this was left on purpose or not.  If we do want to log the port, can we make the log more descriptive?  Otherwise it may just look like:

=> 2013


So false in Python doesn't exist unless you define it per @willbarton.  False is a Python constant. You can check it in the Python console.  Either approach is fine with me.


Remove either "argument below" or "path below".

The venus-include argument below resolves to "../../js/" before becoming an absolute path.


yes :) furtunately, it doesn't matter :-)


Spacing


и как будете стилизовать этот спан как «нет» ? :))
сейчас к нему никак вменяемо не привязаться, равно как и к тем спанам, которые «да»


*good


Tags don't repeat Feature tags on scenarios.

@usat-mweb-menu-sections

On the random subsection scenario use @usat-mweb-random-subsections


If you only `atom` the errors, then this becomes:

```Clojure
(defn set-error [this-individual error-key error-value]
  (swap! (:errors this-individual) #(assoc % error-key error-value)))
```


@englercj yes definitely red flag for me too :/


Suggestion: use active voice

"The 'basePaths' object defines aliases that can be used within venus annotations for brevity/convenience"


в тексте один номер, а набираться будет другой?


whitespace around = symbol


маркер на карту ставится из JS, отдельный жлемент ему не нужен


а что, достаточно неплохо!


Hi,
Need help.
I'm trying to build gobblin and get the error below

./gradlew clean build
Parallel execution with configuration on demand is an incubating feature.
Creating properties on demand (a.k.a. dynamic properties) has been deprecated and is scheduled to be removed in Gradle 2.0. Please read http://gradle.org/docs/current/dsl/org.gradle.api.plugins.ExtraPropertiesExtension.html for information on the replacement for dynamic properties.
Deprecated dynamic property: "hadoopVersion" on "root project 'gobblin'", value: "1.2.1".
Deprecated dynamic property: "pegasusVersion" on "root project 'gobblin'", value: "1.15.9".
Deprecated dynamic property: "classification" on "project ':gobblin-api'", value: "library".
Deprecated dynamic property "classification" created in multiple locations.

FAILURE: Build failed with an exception.

* Where:
Build file '/www/gobblin/gobblin-rest-service/gobblin-rest-api/build.gradle' line: 12

* What went wrong:
A problem occurred evaluating project ':gobblin-rest-service:gobblin-rest-api'.
> Plugin with id 'pegasus' not found.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED

Main build.gradle has following

"pegasus" : [
        "data" : "com.linkedin.pegasus:data:"+pegasusVersion,
        "generator" : "com.linkedin.pegasus:generator:"+pegasusVersion,
        "restliClient" : "com.linkedin.pegasus:restli-client:"+pegasusVersion,
        "restliServer" : "com.linkedin.pegasus:restli-server:"+pegasusVersion,
        "restliTools" : "com.linkedin.pegasus:restli-tools:"+pegasusVersion,
        "pegasusCommon" : "com.linkedin.pegasus:pegasus-common:"+pegasusVersion,
        "restliCommon" : "com.linkedin.pegasus:restli-common:"+pegasusVersion,
        "r2" : "com.linkedin.pegasus:r2:"+pegasusVersion,
        "d2" : "com.linkedin.pegasus:d2:"+pegasusVersion,
        "restliNettyStandalone" : "com.linkedin.pegasus:restli-netty-standalone:"+pegasusVersion
    ]

All libraries are present in .m2/repository

Am I missing something?


i've correct it on my local, i end the blackmarket implement and i push it !


Reformat code.


int idx_min_temp = cursor.getColumnIndex(WeatherContract.WeatherEntry.COLUMN_MIN_TEMP);


This should be fixed in latest `dev` now, I change dit back to use brfs and made fs declared alone. I'm going to raise an issue on the brfs tracker about this and link it here.


LINE 15 - 19 code etiquette by using indentation


I already made pull request for it.


88621024-ElectricLoadCenterStorageBattery-Crash (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1654 of 1654 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)


82420068_GridViewMultiEdit (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


Hi Seth,

Sorry I am just getting to this now. I'm spending the next couple days drastically cleaning up this tool. This import change was fixed in an update yesterday. I greatly appreciate your contribution and hope this tool has been useful to you!


remove the comma after "a grouping of files".

"The 'libraries' config allows you to create a grouping of files that can be passed as an argument to a @venus-library annotation"


Hrm... I didn't read this carefully enough.

Actually, today, transforms expect options to be passed in via `transformOptions.config` (or via `transformOptions.configData.config`.)  If options are passed explicitly to the transform when the transform is created, we should pass these options in via `config` as well.

I would say if options are passed explicitly to the transform programatically, we should not load config from package.json, and instead set:

    configData = {
        configDir: null,
        configFile: null,
        cached: false,
        appliesTo: opts.appliesTo, # ??
        config: opts
    }

This would be consistent with the [documentation here](https://github.com/benbria/browserify-transform-tools/wiki/Transform-Configuration#configdata-object).  Thoughts?


where is this.baseURI set? should be using something like $('body').attr('data-context-url') instead. This whole file should probably be removed though--I thought it was.


избыточный код
мы будем использовать автопрефиксер, соответственно, пишем только то, как нужно по спецификации, без всяких -webkit-


см. имя файла
у нас на проекте IE10+


88621024-ElectricLoadCenterStorageBattery-Crash (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


WinGUI/Settings/151301-0413_Retune_PID2.mws notes:
A good starting point but has these issues:
Yaw is not as strongly "nailed" to one direction. Responds well but drifts fairly quickly. Consider increasing the I value
Twitchy! The quad is very twitchy when windy, the wind also makes it gain altitude, a behaviour also seen when going from moving forwards to "horizontal" - the quad gains altitude when it wouldn't normally do so.
No idea how to fix this particular behaviour: Try adjusting the LEVEL PID values first - remove I value completely? Then INCREASE the P value by 50% and decrease it by 50% and see which one is better. If issue stays during both of these situations, experiment with the RATE PID settings.


not sure I like the oscCollapsed and oscUncollapsed as separate directives


It should be `Kelvin` (capital K) technically :)


http://codepen.io/elijahmanor/pen/Igpoe — как лучше делать иконку «гамбургера»


dims[-1] ?


д.б. подключен один CSS-файл.


I see. If you can get rid of C++ lib wrapper then this is the way to go. In [ModelicaInternal.c](https://trac.modelica.org/Modelica/browser/Modelica/trunk/Modelica/Resources/C-Sources/ModelicaInternal.c#L133) and [ModelicaStandardTables.c](https://trac.modelica.org/Modelica/browser/Modelica/trunk/Modelica/Resources/C-Sources/ModelicaStandardTables.c#L79) of MSL Trunk I use already [troydhanson/uthash](../../../troydhanson/uthash) after @sjoelund pointed to me. See also [here](https://trac.modelica.org/Modelica/search?q=uthash).


The file (Version 1) is a SQL file which contains the DDL to create
Sample database
OPENHMIS2 and the DML to insert the sample data into the tables in the
schema


может, просто .promo?
ofxtv все эти пейдж-контенты?
суффикс -block — вообще не поддается пониманию )))


worth summarising how, or linking to somewhere this is explained in detail?


Hi Scott,

Glad my contributions thus far are helpful. Feel free to ask me if you need
clarification on any of them.

I'm still getting used to the githib work flow paradigm. I can definitely
see the advantage of separate commits, but I still tend to think and work
in a more linear dependent fashion (root + A) then (root + A + B) then
(root + A + B + C) instead of treating them as parallel and independent
branches from a common root: (root -> branch A), (root -> branch B) and
(root -> branch C) which could eventually be merged back into (master + A +
B + C) or any other combination. I just need to force myself to think that
way when I run into a new issue and want to just jump on it and fix it
right away, especially when it's a one-line fix. Anyway, I'll try to do
better with that part. It's a useful discipline.

Regarding the Permutation.java bug, I suspect you are probably concerned
that the method was being called incorrectly from elsewhere and that the
fix I have proposed might be masking another bigger problem. The fact is, I
didn't fix it as a result of any problem that I encountered. I simply
noticed that code while I was single stepping through an unrelated issue
and the incorrect upper limit happened to catch my eye as a common
easy-to-miss faux pas. I don't know of any place that the function actually
gets called with the invalid value, but being a bit of a purist, I decided
to override the old "If it ain't broke, don't fix it." adage and make the
correction anyway to prevent a potential problem in the future.

Regarding ANTLR, I'd definitely like to see it integrated into the build as
you suggest. In fact, since I was unfamiliar with it, I downloaded the
latest version (4.5) along with supporting plug-ins for eclipse, netbeans
and intelij. I've already spent several hours going through the whole ANTLR
2.0 tutorial at http://javadude.com/articles/antlrtut/ as well as the
entire ANTLR 3.0 video tutorial at http://javadude.com/articles/antlr3xtut/.
Both of them are developed by the same guy who does a really excellent job.
Unfortunately, I don't see a 4.0 tutorial by him. Having worked briefly
with lex and yacc in the '80's, and having written a small recursive
descent language parser and interpreter for SNMP from scratch in C and C++
around the same time, I am familiar with the concepts involved, so I have
been playing with it a good bit this past week. I did note that the ANTLR
grammar itself has evolved with each major release and so they are not
completely backwards compatible with prior major releases. That said, I
wouldn't mind helping to migrate zomic to a newer version (either 3.x or
4.x) if you'll help with the regression testing. On the other hand, I know
that the  if you want to leave zomic alone and only do small maintenance
fixes (like the EOF fix within comments) then that's your call and I'm OK
with it either way. Vzome currently uses ANTLR 2.7.2, but since it's so
old, most of the info I have been able to find online is not directly
applicable. Also, many of the available plug-ins for visualizing the ASTs
(including all of the ones I have installed) are designed to work with 3.x
or 4.x. Finally, you mentioned that you are working on a Python API for
vzome, which presumably would provide all of the zomic functionality and
more. If you think our time would be better spent working on that, I'd be
glad to do that instead. In case you'e interested, I came across a library
of grammar definitions for different languages including python at
https://github.com/antlr/grammars-v4. Note that these are all written for
ANTLR 4.x, so there's one more reason to upgrade. I suspect that zomic
could remain static using the ANTLR 2.x library and the new Python API
could coexist with it in vzome-core using ANTLR 4.5, but I'm not sure if
that's so, or if it's a good idea either. Anyway, I still have several
other questions and ideas and suggestions for zomic (and/or Python) that we
we'll need to discuss some other time.

On Thu, Mar 5, 2015 at 2:50 AM, Scott Vorthmann <notifications@github.com>
wrote:

> All good stuff, David. I'll probably take most of these as manual fixes; I
> want to think a little more deeply about a couple of them, as the core
> defect may be elsewhere.
>
> One hint for future reference: you can make an individual commit for each
> separate bug fix. This would be much preferable for me, since I could pull
> then selectively merge certain commits using git, rather than copy-paste.
>
> Another request: please create a defect for the Permutation.java bug, and
> explain how to reproduce it, attaching a vZome file if necessary.
>
> I'll try to reproduce all the others, and I'll let you know if I fail.
>
> Finally, yes, I'll verify how you incorporated the EOF fix into the ANTLR
> grammars, and while I'm at it, I'll work out how to make ANTLR part of the
> build, even if an optional part.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/david-hall/vzome-core/commit/c9d4e7cb6dd0183b787fec6a1d32362be0f289fe#commitcomment-10046183>
> .
>


Should I use std::assert or ARG_ASSERT here?


The text of the spec does not match the expectation. Write another spec for `it "will require a program_name"`


тут и далее используете модификатор без класса, который он модифицирует
и еще:
- это не img
- это ссылки
- требуют текстовых узлов


модификатор без класса, который он модифицирует


Twitchy improvements: Try decreasing the P value in the RATE PID. Try increasing the I value.


new tool, not format.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1145 of 1146 tests passed)

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


I do not know why it shows System Administrator push this. It is me.


it should be `if (dim > 0) return;`


@gatlingfun
Did you mean to send a pull request?


Good


86046200_CondenserToZone (Nigusse) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


КОДГАЙД!


Venus searches for a file named 'config' in the '.venus' directory then proceeds to walk up the directory tree looking for other '.venus/config' files.


just make sure this doesn't get checked in for the final version


I guess @GoodBoyDigital will want to add this and the change a bit above back in again :)


Also removed the excess CSS on the game view page (that was initially put in to make the sinatra game app look like the rails app for waffle demo).


есть ряд недостатков, самые яркие я откомментировал.
предлагаю созвон.


No, it works fine here.You might need this kernel patch: https://github.com/CyanogenMod/android_kernel_samsung_msm8930-common/commit/2fe34fc0ba0c3e06265d99e99c511572494794f5


What do you think of 'Intention of this text is to document already know bugs and how to work around them.'?


Also removed default styling for every link in the application and applied a class tag for it. This will make styling other links in different contexts possible.


nitpick: lowercase u in Use


'wrong translations of them' maybe?


I disagree that we should necessarily make this a generalized model. Given the way I modeled this, it would be pretty easy to do.

However, we would need to add a new M2M to Event for each type of form we want to support. And we would have to use an M2M through to properly handle ordering.

Mostly, though, I think it's incorrect and inefficient to model this as an M2M, since no form will ever be attached to more than one event. I could do it as a GFK, but it won't actually ever be connected to anything *but* an event, and also we'd still need to store the form type somewhere.


So in short, I think this is easiest & fastest.


You have the wrong email address configured for git.
If you check https://github.com/Exartu/Exartu-CRM/commit/9e56de362017d497633fdc79d7659c9176881e87.patch will show you the email that git used for this commit


Define a library (```mocha```) inside the libraries object. The library(```mocha```) should include an array called ```includes``` that contains an array of filepaths.


That won't be a consideration for a looong time.


//Comment more code


Ow awesome! I was thinking of creating something like this myself as pull request!


That bug wasn't related to previous bug. It was relating to finding a user by ID that didn't exist when checking if user existed before creating the new user. The response builder returned entity not found in query exception as I did not catch that exception and deal with that exception. Same would of happened when logging in with an incorrect userID so this bug would have caused another issue later on. Resolved now.


Why will no form will ever be attached to more than one event? Someday in the far future, forms should belong to users or orgs, not events, I think. So that you don't have the remake your form next year.


> The closing brace/bracket/parenthesis on multi-line constructs may either line up under the first non-whitespace character of the last line of list [...] **or it may be lined up under the first character of the line that starts the multi-line construct**


Thanks for pointing that out. We probably don't need that. Will fix it in the next patch.


```yml
degree:
  title_html: 'Please choose a <b>degree</b> of your work'
  hint: 'You must have choose a degree for your work'
```

Add a new entries for program name:

```yml
degree:
  title_html: 'Please choose a <b>degree</b> of your work'
  hint: 'You must have choose a degree for your work'
program_name:
  title_html: 'Please choose a <b>program name</b> of your work'
  hint: 'You must have choose a program name for your work'
```


nitpick: "that can be included" verses "that can included"


Fixed.


why did you remove sabermod?


Yeah, I guess this is because I use root user to push.


'and it will not work' maybe (removing of the last 'correctly' as it's repetitive)?


where is this tar downloaded?


Nit; The second `<a>` needs a `/`. :)


Don't overload the translation for degree.

```yml
degree:
  label: "Add <b>degree</b> for your %{entity_type}"
```

```yml
degree:
  label: "Add <b>degree</b> for your %{entity_type}"
program_name:
  label: "Add <b>program name</b> for your %{entity_type}"
```


If we go this route, I'd suggest moving the definition up to the "Terms" section, and just referencing it here.


Separate degree and program_name validation into separate concerns.


opcodes are correct but structures needs to be changed aswell before this can be applied


Coverage fails for me...
`Coverage failed. Falling back to standard unit tests.`


:+1: Nice change


I want to describe purpose of this document. That it is not just bugs we know, but it also contain knowlegde how to deal with it.


and Java Delaunay


That was an easy adaption. I am impressed! LGTM.


This is now fixed.


right thanks


we have a bug for making this message better, please install coverage3 or so.


THERES a HACKER ONE HACKER asswagaming


Guess it should be enabled by default when we merge it to master.


Yes, because it may significantly reduce the number of send() / recv() syscalls (which result in higher number of network packets and, consequently, higher network overhead) on loads with 100-1000 concurrent users. See the numbers (obtained via ConnStats):

Default FlushDelay:

    BenchmarkRealApp100Workers-2  Client: reads=452, writes=436. Server: reads=438, writes=450
    BenchmarkRealApp1000Workers-2  Client: reads=194, writes=183. Server: reads=173, writes=202

Disabled FlushDelay:

    BenchmarkRealApp100Workers-2  Client: reads=6579, writes=6961. Server: reads=6303, writes=7309
    BenchmarkRealApp1000Workers-2  Client: reads=392, writes=948. Server: reads=343, writes=996


I would expect that to be handled by duplicating forms when you duplicate an event (the same way you'd duplicate items and discounts.)

If you used the same form, then you would actually modify the past event's form when editing the form for the current event, which is an unexpected (and potentially confusing / data destroying) side effect.


you are right, I rewrote this sentence and do not fix all places. Thanks


This is not the proper place to report a hacker, and commenting on a commit is not the proper way to report an issue. Please read the readme on the main issues page.


nitpick: should run be in the past tense? i.e. When venus is ran it looks for blah blah blah


https://github.com/Knox-/TSCalculator/blob/master/TSCalculator/TSCalculator.cs#L71

And the same lines in the other methods: It requires 2 parameters, so it should check (args.Parameters.Count < 2), not (args.Parameters.Count < 1). Ideally should be (args.Parameters.Count != 2), though.


Please use the gpr_int64 type, and remove this #ifdef


some tests failed:
failure: http://build.servo.org/builders/linux2/builds/947


@lighta mind to comment what you still wanted to change or is everything fine by now?


I added in the uniform change, didn't realize that was added. But the above bits are fixed now and work properly with brfs, so no one should need to change it.


Prefer:

if (success) {
  ...
}


Remove newline


Mmm fair. And if we were super cool we would give you a list of your old forms that you could _copy_


Add space

(This code would benefit with a pass through clang-format)


Add space before private:


I don't know where it is


Nice catch. We should check the rest of the code base for calls to `slice`.


`lambda x:` takes a parameter, soI think it should be something like `(self)` at the end instead of `()`.


develop (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


if you change the opcode, update the structure too !


I'll have to double check, but I don't think all file-like objects derive from `IOBase`.  e.g. a file returned by `open()` does not, but `BytesIO` does.


Notice that participant_note_path has 2 dynamic segments, :participant_id,
and :id, meaning it needs 2 arguments.

On 6 March 2015 at 18:18, Eric Wiseman <notifications@github.com> wrote:

> Thanks Eliot. I'll keep working on it. I realize that @note_path was not
> a good solution, but it was the only thing that yielded a result (i.e. it
> destroyed the participant) : )
>
> I did look at the output from rake routes and what I saw for notes#destroy
> is participant_note and /participants/:participant_id/notes/:id(.:format)
>
> This was my first attempt:
>
> participant_note_path(@participant)
>
> followed by several iterations of the above that also did not work.
>
> I'll continue to work on it and let you know what I come up with . This
> route stuff is working me over. Sorry for all the question about this....
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Ewiseman/Seva/commit/f0d7397ffb89d412b6081f1b01e9209ee178be8d#commitcomment-10076830>
> .
>


Eventually we'll need to split large messages into O(8kb) blocks to avoid fragmentation issues.
TODO is ok for now though.


@diurchenko the problem that I see is that the intra region stuff doesn't need to get mixed up here at all. The intra region use case is separate from everything. In that case we don't care what the operation mode is, we just need to retain the attrs.


Oh this is gonna be fun for everyone outside of our glorious timezone CEST.


Oops this doesn't support CentOS


This is not the original script, I've added some stuff.


Ok, that's right. I was just pointing to the repetition of 'document'.


good catch, should just be `lambda: 'objects'`


int radius = 15;
int direction = 1;
int direction2 = 0;

float x = 250;
float y = 250;

ArrayList<Particle> poop = new ArrayList();

void setup() {
  size(500, 500);
  ellipseMode(RADIUS);
  for (int i=0; i<50; i++) {
    Particle P = new Particle((int)random(width), (int)random(height));
    poop.add(P);
  }
}

void draw() {
  background(255);
  fill (0, 175, 255);
  smooth ();
  noStroke();
  render();

  for (int i=0;i<poop.size();i++) {
    Particle Pn = (Particle) poop.get(i);
    Pn.display();
    if (dist(x, y, Pn.x, Pn.y)<radius) {
      poop.remove(i);
      radius=radius+1;
    }
  }
}

class Particle {
  int x, y;
  Particle(int x, int y) {
    this.x = x;
    this.y = y;
  }
  void display() {
    noStroke();
    fill(#FCA900);
    ellipse(x, y, 10, 10);
  }
}

void keyPressed() {
  if (key == CODED) {
    if (keyCode == LEFT) {
      x = x - 10;
      direction = -1;
      direction2 = 0;
    }
    else if (keyCode == RIGHT) {
      x = x + 10;
      direction = 1;
      direction2 = 0;
    }
    else if (keyCode == UP) {
      y = y - 10;
      direction = 0;
      direction2 = -1;
    }
    else if (keyCode == DOWN) {
      y = y + 10;
      direction = 0;
      direction2 = 1;
    }
  }
}

void render() {
  for ( int i=-1; i < 2; i++) {
    for ( int j=-1; j < 2; j++) {
      pushMatrix();
      translate(x + (i * width), y + (j*height));
      if ( direction == -1) {
        rotate(PI);
      }
      if ( direction2 == 1) {
        rotate(HALF_PI);
      }
      if ( direction2 == -1) {
        rotate( PI + HALF_PI );
      }
      arc(0, 0, radius, radius, map((millis() % 500), 0, 500, 0, 0.52), map((millis() % 500), 0, 500, TWO_PI, 5.76) );
      popMatrix();
      // mouth movement //
    }
  }
}


Case in point https://travis-ci.org/bliksemlabs/rrrr/builds/53379165

Remember the time i kept bitching for like an hour about strtoepoch and timezone bullshit? This was IT ;)


Test


holu


Например, когда вводим символы не входящие в HEX формат (G-Z). А также, в том случае, если длина строки != 6. Получается, что в этих случаях побитовый сдвиг срабатывает, но не корректно.


Note: This is generated for testing. On production we use a different salt which has not been disclosed publicly.


LGTM


so maybe use `Intention of this text is to document`


@MikaelSmith minor issue with commit message ... it says `--genconfig all` instead of `--configprint all`


I believe this group should be "apache" on CentOS guests.


Okay, I'll keep in mind that so next time when I'm updating opcodes I'll try to update structure too.


new filw


@michelgomessilva do you have any examples of using OrmLite with jsonb on Postgresql 9.4?


I always lean to the side of evaluating truthy statements (see idiomatic.js) but I can see how it might be confusing here. What if in this instance we include the argument keyword when utilizing the function:

```
share(post.title, hide_header=True, { ...twitter stuff... })
```

It's a little longer but it makes it clear what you're doing.


Если честно, до конца не понимаю, в каком случае стоит сносить и выравнивать по знаку ":", а  каких стоит писать в одну строчку. Xcode, в случае длиной строки, автоматически переносит длинные строки, но что является верным – непонятно.


I guess we can now say that time was on your side... :P


trop de probleme avec ce fdp d'enculer


I can't reproduce the problem now.  I'm working on master, but I munged the `mica.web` package names and can do a kadi event query without `kadi.urls` ever being imported.  If you have a demonstration of this can you open an issue?


I think yes. You could use «intention» or «the goal» maybe.


why do we need this? are they inside **build** dir?


Sure, I'll keep that in mind so next time. So do you want me to close this PR and make new one with updated structures too or ?


oh, there's a 1.7.38 now!


There are lots of lines like this; I believe they are caused by tabs vs spaces. Since the rest of the codebase uses soft-tabs, please use soft tabs.


88621024-ElectricLoadCenterStorageBattery-Crash (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1654 of 1654 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)


**local.properties** are excluded recursively, starting from project root


I removed almost all of the old command line switches as I'd never implemented them for the tex4ht version, so they didn't do anything once latex2html was gone, and nobody seemed to have cared. I think the aux one just copied in the current aux file and used that?


**idea** folder fully exluded


Глупо, конечно, но немного насторожила надпись "Assigning retained object to weak property", с учётом того, что ARC включено.


How about using [`assertRaises()`](https://docs.python.org/2/library/unittest.html#unittest.TestCase.assertRaises)?


The resolution isn't correct with this. Height, Width works correctly.


This commit actually made by Peter, if it matters.


This is a great catch, thanks! Merged in.


ack


so just to be sure - anyone (authorized or unauthorized) can checkout? no need for a resolve here.


This is great @rbakr and super helpful. Would it be worth mentioning the static property too? I can always add it later if you don't get around to it.