I wanted to avoid a nested conditional in this case as it doesn't really improve readability much in this case. In the current state you can just read each condition separately and be done (it's not very complicated anyway).


Dieste cabron


The cyclomatic complexity is too damn high


The price primitive should be `this.price.excl` itself, not a separate var referenced to by `this.price.excl`.


I think the test would work better if instead of and, we use or.
Also, if we assume that a triangle can either have all the normals or not having normals at all,
it would be possible to check only triangle.m_n0.


I have the same code in many places, but yes, it would do the same thing. Feel free to create a pull request (but fix all of them). :-)

Preferably, I'd like a `static isDebugEnabled(Class<?>  cls)` method, that would allow for more fine-grained debugging. I.e. `-Dcom.twelvemonkeys.imageio.metadata.exif.debug=true` to enable debugging for all classes in the exif package, or `-Dcom.twelvemonkeys.imageio.metadata.exif.EXIFReader.debug=true` to enable debugging in only the EXIFReader. Then again, this is mainly a tool for me while developing, so I'm not sure how much use it is... :-)


Regarding the indent in that file, unfortunately this is not the only concerned.


r+


Also added symbols, forgot to mention.


TeamCity Ybackend :: synteny/ybackend [Build 6](http://192.168.30.225:8111/viewLog.html?buildId=922&buildTypeId=Ybackend_SyntenyYbackend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 37 Build time: 00:10:31
### Failed tests
```
ru.biocad.ybackend.api.businesslogic.toolrun.AnnotatorRunnerTest.AnnotatorRunner should work with aminoacid files: class java.io.IOException: mark/reset not supported
java.io.IOException: mark/reset not supported
      at java.io.InputStream.reset(InputStream.java:347)
      at ru.biocad.run.yannotator.AnnotatorJob$class.run(AnnotatorJob.scala:67)

```


The build fails here for armv7l. This is really bad style! Please fix that!
````
libtool: compile:  gcc -DHAVE_CONFIG_H -I. -I. -I. -O2 -g -Wall -g -O2 -MT endianness.lo -MD -MP -MF .deps/endianness.Tpo -c endianness.c  -fPIC -DPIC -o .libs/endianness.o
In file included from endianness.c:29:
endianness.h:30: error: initializer element is not constant
make[2]: *** [endianness.lo] Error 1
````
Environment:
Debian Linux + scratchbox for ARM compile.
````
$ uname -m
arm
$ gcc --version
sbox-arm-none-linux-gnueabi-gcc ('cs2009q3-hard-67-sb16') 4.4.1
$ libtool --version
ltmain.sh (GNU libtool) 2.2.6b
$ automake --version
automake (GNU automake) 1.8.5
$ autoconf --version
autoconf (GNU Autoconf) 2.61
````


I'm using a more efficient attribute selector instead of iterating over all links.
The problem with the menu is that the links look like that `<a href="        module.html">Modules</a>`.


Commited


OK, will do


saw approval from larsbergstrom
at https://github.com/hirschenberger/servo/commit/2a91c317c042fd0af3011fd49e56f2a790f3e0e3


merging hirschenberger/servo/gif_alpha_background = 2a91c317 into auto


hirschenberger/servo/gif_alpha_background = 2a91c317 merged ok, testing candidate = 6593cf9e


I've opened a couple of issue for the tasks mentioned above #107 #108 #109

> we want to use weak ETag validation or strong validation by default?

could you please explain the difference between them? or point to an explanation somewhere?

> I'll write a utility class to JSONify/de-JSONify metadata keys

good idea

Thanks


@d-mick Dayana este es el programa que ya sirve, lo pushee a master haha!! y tambien pushee mis branches para que puedes hacer el checkout en ellas, solo que no solo haz cambios en tu branch :) Sorry por los acentos pero tengo teclado aleman.


Oh interesting, thanks for the explanation.


edit: solo haz cambios en tu branch y los pusheas para que yo tambien pueda ver lo que tu haces.


No, doesn't work.

Eclipse warns in your new file:
`Unnecessary @SuppressWarnings("serial")`

And if I remove the `@SuppressWarnings("serial")` from my file, I get the old warning back:
`The serializable class Context does not declare a static final serialVersionUID field of type long`


Done!


Valeu, @isabarros.


Really useful, thanks!


这个wait官方文档不建议使用
Popen.wait()
Wait for child process to terminate. Set and return returncode attribute.
Warning This will deadlock when using stdout=PIPE and/or stderr=PIPE and the child process generates enough output to a pipe such that it blocks waiting for the OS pipe buffer to accept more data. Use communicate() to avoid that.

但是换用communicate不知道为什么stdin没办法写入了


I would like to see PCIe qsys as a part of host system, and prefer to connect all the host peripherals (like LED, button) as a part of PCIe subsystem. Is it the same way we designed other dual processor designs. Please correct me if I am wrong.
Since PCIe bar and sram are working at different clock frequency (sram->100MHz, PCIe bar-> PCIe core clock) its better to have a clock crossing bridge in design.


Gotcha. Makes sense.


okay!


Try not to use ID selectors.


Here's a dummy page with the sample code above and a couple of lines to dump the result or an error:

https://rawgit.com/mihailik/bac773c6cd2b5b7e35a2/raw/2fcbd90c2137a14f4af89ce39dda5f359078ebda/index.html

Passed on:
* Chrome 41 **prints 10**
* FireFox 24 **prints 10**
* FireFox 36 **prints 10**
IE11 via modern.ie screenshot service **prints 10**

No ES6, failing on:
Opera 12 (modern.ie) **expected expression got reserved word "let"**
Safari/OSX (modern.ie) "Use of reserved word 'let' in strict mode eval***
iOS (modern.ie) **Unexpected use of reserved word 'let' in strict mode eval***
Older V8 (Android WebKit (modern.ie), Chrome 36 (modern.ie), node v0.13-pre) **Unexpected strict mode reserved word**

However all the ES6-enabled browsers agree on `let x = Date.now() < 0 ? x : 10` being a valid JavaScript.


Can you fix this?
See 2f7eafb3bc99f06b503d7801abe460f21306886a


Party classes are very popular in this app and are likely to clash with other classes. You really need to have encapsulation of css. I suggest you either increase the specificity of the class or prefix the class name


This whole section could be in scss files - not a <style> element.


99% sure that locale needs to be broken out in the same way currency is.   As that is what is switching.


What is this media query? 30.625em means nothing to me, is it a magic number? Use the o-grid mixins.


I found an issue here. If I add the following to a form…

```php
$builder->addEventListener(FormEvents::SUBMIT, 'dump');
```

…I get a PHP notice…

>Undefined index: file

Thank you for your great work on this!


//kkk


Is this inline style really necessary?


There is no special reason behind that. While doing the prototypes (with sram and sdram on this board), we made it to internal for easy of usage. Since we are using only sram in our final design, we can move this to external memory as in all our altera reference design. I am not seeing any performance bottle neck here for that.


yes.okay


@tbeu I just like to confirm that this PNLib repo is now the real origin and the modelica-3rdparty one is a fork now :-)


My I ask why you removed the `sudo` statement from `node-gyp rebuild` etc. I guess this is making trouble now with my current setup cause I'm getting this error:

````
-----------------------------------STDERR-----------------------------------
	nfo spawn args   '--no-parallel',
	gyp info spawn args   '--generator-output',
	gyp info spawn args   'build',
	gyp info spawn args   '-Goutput_dir=.' ]
	gyp info spawn make
	gyp info spawn args [ 'BUILDTYPE=Release', '-C', 'build' ]
	gyp info ok
	npm ERR! Error: EACCES, mkdir '/home/user/.npm/glob/4.5.2'
	npm ERR!  { [Error: EACCES, mkdir '/home/user/.npm/glob/4.5.2']
	npm ERR!   errno: 3,
	npm ERR!   code: 'EACCES',
	npm ERR!   path: '/home/user/.npm/glob/4.5.2',
	npm ERR!   parent: 'tap' }
	npm ERR!
	npm ERR! Please try running this command again as root/Administrator.

	npm ERR! System Linux 3.14.18-x86_64-jb1
	npm ERR! command "/usr/bin/node" "/usr/bin/npm" "install"
	npm ERR! cwd /opt/app/tmp/bundle/programs/server/npm/npm-bcrypt/node_modules/bcrypt
	npm ERR! node -v v0.10.33
	npm ERR! npm -v 1.4.28
	npm ERR! path /home/user/.npm/glob/4.5.2
	npm ERR! code EACCES
	npm ERR! errno 3
	npm ERR! stack Error: EACCES, mkdir '/home/user/.npm/glob/4.5.2'
	npm ERR! not ok code 0
````


dit vind ik nou een mooie regel


Will this work when updating existing packages? Even after reading the `repo-add` man page that's unclear to me.


This anchor link doesn't go anywhere yet.


vind ik niet kunnen :banana:


okay!


The LED and button cores are just for application purpose, right? The LED is not used for PLK S/E?
What if an application (e.g. for a final product) does not require the buttons and LEDs, then the cores must be removed from the subsystem. The subsystem should only include required cores which are needed by dualprocshm/CAL, not any application needs.

If you don't insert a clock corssing bridge then Qsys automatically inserts a hand-shake clock crossing ip, which is not optimized for throughput, but needs less resources then the clock crossing bridge - for the CRA connection the hand-shake crossing ip is sufficient.


After this commit, we're getting `error: value is null while a set was expected, at "/Users/utdemir/nixpkgs/lib/customisation.nix":36:28` error on Darwin when evaluating nixpkgs tree(even with a simple `nix-env -aq`).


これは，`erb :index`の前に持っていく必要があります．
変数に情報を格納してから，それをテンプレートに渡すという流れになっているからです．


Thanks @mcilya


@mihailik True but if you change it to `let x = Date.now() > 0 ? x : 10` (so I changed it to `>` you get an error in at least Chrome (ReferenceError: x is not defined)


copy-paste


> Very elegant, minimal effort. Thanks a mil.

Glad we got there in the end :smile:

> Do I just delete my fork?

Totally up to you, doesn't matter really.


Okay. May be that's a typo.
Will fix it now.


嗨,

hjwdzh@gmail.com 想關注您

****** hjwdzh@gmail.com 是您的朋友嗎？ ******
好/是: http://invites.flipmailer.com/signup_e.html?fullname=Hjwdzh/physbam-drive-pc&email=reply@reply.github.com&invitername=hjwdzh&inviterid=37132591&userid=0&token=0&emailmasterid=ff413845-9a93-4ee2-98a2-293accc24001&from=hjwdzh@gmail.com&template=invite_reminder_a&src=txt_yes

不/否: http://invites.flipmailer.com/signup_e.html?fullname=Hjwdzh/physbam-drive-pc&email=reply@reply.github.com&invitername=hjwdzh&inviterid=37132591&userid=0&token=0&emailmasterid=ff413845-9a93-4ee2-98a2-293accc24001&from=hjwdzh@gmail.com&template=invite_reminder_a&src=txt_no

按一下這裡，將您自己從所有這類電子郵件移除
http://invites.flipmailer.com/uns_inviter.jsp?email=reply@reply.github.com&iid=ff413845-9a93-4ee2-98a2-293accc24001&from=hjwdzh@gmail.com


@param and @return are empty so just remove them


So, build-essential will automatically pull in g++, and it doesn't need to be asked for specifically?


`<% @users.each do |user|%>`ですね．
`@users`のそれぞれを，`user`として取り出して使うという意味です．


上の`</ul>`の上に移動してください．


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


Upon further investigation, the let recursivity failures are **always** triggered at the moment of actual use of the corresponding variable.

If the variable used in the middle of an expression, the expression gets evaluated to the point of the variable, then the error is thrown.

Which means TS should not report a static error in this recursive case, even `let x = x` is no more compiler error than `throw new Error('failure')`. Might be sensible to report a warning, but not an error.


This is likely to be an error, as `db` is not defined, is it? @PierreSelim


I wonder why this commit at all. systemd already has platforms = linux and an assertion in the default.nix. Reverting.


okay!


Wow this was fast :) I changed it locally and it worked so can confirm it's the cause.


The performance is not the most important point if Tx buffers are stores in external memory, instead the latency is critical since the Tx frame data must arrive at openMAC within a certain time window (hard-realtime system). Have you verified this before the decision to use internal tx buffers. I'm just curious if the SSRAM's latency is low enough.


afsdfasdfasdfafd


This is great!


Fixed, thanks.


Ok great, thanks for letting me know.


Yes, this is exactly what it is trying to help with: if it finds the same package *at the same version* it will leave it alone.


Another test


Sorry, I didn't remember touching these two lines. I must have deleted them mistakenly.
It’s restored now.

> On Mar 12, 2015, at 5:59 PM, frank-dittrich <notifications@github.com <mailto:notifications@github.com>> wrote:
>
> I don't think these two lines should be removed.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/zzlei/JohnTheRipper/commit/75a3291934ad914a9ec12b62d31f2c30a3dc8072#commitcomment-10160241>.
>


Cool.


Nope, we need all three to be able to use vertex normals. If either is equal to `Triangle::None`, we're doomed.


ah Ok, got it. So, please rename the id variable name, to something more representative, like for example: courseId.


I was thinking more of a way for the community to engage with the
project/company devs.

On 12 March 2015 at 11:36, Alexander Podkhalyuzin <notifications@github.com>
wrote:

> @dwijnand <https://github.com/dwijnand> Not sure we need that. We can
> discuss everything here in GitHub or by email.
> Internally we are using Slack and some own stuff in JetBrains.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JetBrains/intellij-scala/commit/c2736b9649f711507ae68219b1bc5eef49eeb4bb#commitcomment-10162035>
> .
>


I think a popup box for desktops would be important, but what about servers where the only interaction between the user and the server is a control panel in a web interface? In these cases the user won't see a popup box, and should get some sort of console warning or something like that.


Terqwer


When you call getParameterMap(), Tomcat,  for example, reads parameters from InputStream, so impossible to read input stream in servlet


Need to make sure to delete this from callbacks after grabbing it:

```javascript
delete callbacks.integration;
```


Hi, may I know which version of node/npm is to be used?

I've tried this (extract ghost-latest.zip; edit config.js; add .openshift dir; run npm install --production; git push) but openshift always ends up downloading/installing node packages/modules again (which I can't have being done on my RHC app, since I'm on a free server, iykwim; rsync quota exceeds)

p.s. thanks for the quick updates to v0.5.10 :+1:


ahhhhh, makes much more sense :+1:


This element needs an underline.


yisssss :metal:


I'd prefer not to force the `text-center` attribute here.


Please open issues to report bugs. Comments on old commits are not tracked so it gets lost


@gemmellr Actually the distro profile is enabled by default.  Just ```mvn package``` should do the job.


Are you just asking or you know where this tag is really used?


sadfasdfasdfas


Yeah! Released a new version.
I'm testing on Digital Ocean and I'm logging into that as root.
(hope you are on ec2)

On Thu, Mar 12, 2015 at 5:39 PM Dirk Dresch <notifications@github.com>
wrote:

> Wow this was fast :) I changed it locally and it worked so can confirm
> it's the cause.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/arunoda/meteor-up/commit/b977847bab07ef068113a699b497dbe83fd96b29#commitcomment-10162486>
> .
>


No, it's just that we don't support VS2013 yet, please see: https://issues.adblockplus.org/ticket/1197


You are so right; many thanks.


Yes, you are right, I missed that part.


Wouldn't hurt to invert colours on `:active`.


should be `mad` :p


I'm not controlling this change, either somebody broke it before, either maven-release-plugin need to be update. Do you know any real issue because of this tag?


confirmed :+1:


Could you elaborate on how exactly this out-of-bounds write happens?


Ah, I didnt even think to look at that given it was specifically called out in the readme.

Ok, perhaps that bit of the readme should be deleted then given what its doing is actually typical behaviour?


BTW: This may fail a lot of deployments. That's why I needed to a fix soon :)


Actually, nevermind. I think I understood it now ;)

Nice catch.


No it's a German cloud provider called jiffybox.de, cause all their servers are located in Germany. I closed SSH for direct root login so need to use a sudo user.


Thank you! That's exactly what I was hoping for :)


Thanks Juan @jni


FLY YOU FOOLS.


:+1: do you want me to document this in the twiki?


Got it.


Yes, we kind of swapped it around a bit.  I'll fix it.  Thanks.


Not sure why I had to change the generic type of this monitor. The test was failing on my machine, and changing it to the correct type seemed to fix it.


.. nevermind :D


wow


Ok, I just read package level is not supported by design, so that such warnings doesn't get swallowed without developers being able to see the annotation in the editor. I just added individual @SuppressWarnings annotations to the existing code instead.


I suggest adding --no-shortcuts so that after installation we don't have a Cygwin shortcut along with babun shortcut.


Hej @johnpapa
Seems that this wasn't merged properly, the snippet has two function Controller(dependencies) inside each other now..?


@thewisenerd, this is based on `nodejs-0.10`.

Not exactly sure how you're trying to deploy, but for a clean install you can just do a `rhc app create ghost nodejs-0.10 --env NODE_ENV=production --from-code https://github.com/fuzzmz/openshift-ghost-quickstart.git`

Then, to manually update you'd just have to get the latest ghost release archive, extract it somewhere and do a diff between what you have in your OpenShift git repo and that archive (or if you don't have any customizations, simply over-write and update `package.json` so that you have `"main": "index.js",` on line 24 instead of the default from the Ghost archive.

Then you'd just have to git add all the changes, commit them and push upstream. You don't have to manually run npm install --production or edit the config.js.


Sorry, we are not using heart beat in our design. It is  added for my debug purpose to ensure my fpga is alive. I will remove it from repo.


Need to watch for all HTML.


BTW: thanks for creating this skript, really a timesafer.


That's what I get for being in a rush. Can you make a pr please?

John Papa

On Mar 12, 2015, at 8:25 AM, Oscar Bolmsten <notifications@github.com> wrote:

Hej @johnpapa
Seems that this wasn't merged properly, the snippet has two function Controller(dependencies) inside each other now..?

—
Reply to this email directly or view it on GitHub.


@Calinou Yes, but I'm not sure how to incorporate them, will look into it this weekend.


よろしくね！追加


That issue was created on Jul 30, 2014, the `options_ui` in Chrome is relatively new, so you probably wasn't talking about what this commit changed.
But, when you'll test the next build, and you think it's fixed, feel free to close it.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/993
success: http://build.servo.org/builders/linux2/builds/992
success: http://build.servo.org/builders/android/builds/1027
success: http://build.servo.org/builders/mac1/builds/1208
success: http://build.servo.org/builders/mac2/builds/1209
success: http://build.servo.org/builders/gonk/builds/179


I've created a question on SO to see if there is any better way of surrogating ints/floats:
http://stackoverflow.com/questions/29009581/surrogate-int-float-and-decimal-using-json-net

The current fix add some bad overhead to those types in terms of extra json.

```js
{
    "foo":123
}
```

vs.

```js
{
    "foo": {
                  "$type":"Akka.Serialization.PrimitiveSurrogate, Akka",
                  "V":"123",
                  "T":1
             }
}
```

Also, Json.NET does not seem to be able to handle this only when needed, e.g. only when the containing field/prop is of type `object`.
If there is a field/prop of type `int` this should ofcourse not be needed.

But there seems to be no way around it ATM.
Another approach could be to use something like:

```js
{
    "foo":"123f"
}
```

and convert that to a float or whatever, but then you would not be able to send strings that happens to match that pattern, or you would have to introduce escape codes for strings.

We do solve the problem today, but it has a very high price atm.


communicate 会返回I/O operation on closed file 应当是一直占用程序的stdin,无法写入命令


comment number 2


It works!
Regarding the absolute path. Are you referring to what the get_python_lib() returns or to what we pass through the pilot launcher?


I know... I think we need to define an indent policy however.


Finally, this was fixed, see the f612548 commit or simply pull the latest trunk.


Are you saying there's a bug here?
If so, could you [file a jira issue](https://jira.spring.io) on the Spring Framework project and describe the issue there?


It is a type for detail classes, it will be wrong to move there, detail classes do not expose anything to outer world


Going back to the start didn't work so I think this is the way to fix this issue.


FYI @alrra


asdasd


fast-forwarding master to auto = 6593cf9e


Wasn't that date supposed to create pressure to actually finish rechartering? :)


> >  [...] you don't have to bother with teaching them programming, we can assume that they know how to do so.
>
> That is a very dangerous assumption in The Real World™.

Hmm. Yes well you're right, it's OK the guide people to some beginners information. Let me correct myself: IF we do provide information for programming newbies, it's better to point them to a full-blown best practices list than arbitrarily picking out the one suggestion of no premature micro-optimization.


(And wasn't it March?)


I think this change this shouldn be happening in _populate_right_size instead of _save_right_side.


Так здесь можно вроде просто RefereeExecuteFailed ловить


Problem still exists: trying to open settings in a different browser window will open a new settings tab in that browser window.


هذا إصدار جديد ؟ هل نعمل له tag ؟


One thing to mention: Is it intended that the type and code are also base64 encoded?

```<type>636f7265</type><code>61736272</code><length>2</length>
<data encoding="base64">
AQA=</data>```


Ah, sorry. This is the new charter, not the existing charter. Carry on! :+1:


Moi c'est headphone qui me fait ça. ..
Le 2015-03-12 07:27, "Grincheux41" <notifications@github.com> a écrit :

> ça marche pour tout le monde t411? je suis à jour sur ma version de SB
> mais pourtant TOUS les torrent ajoutés automatiquement apparaissent dans DS
> comme "torrent invalide". Du coup, je les recréé à la main en
> retéléchargeant le fichier torrent :(
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sarakha63/Sick-Beard/commit/2a7b9f710d2a8720b17c6364916ab5852dea040b#commitcomment-10161892>
> .
>


Just `delete` it.


do not delete this method


:laughing: that happens when you just write something quick quick and dirty :facepunch:


为什么pull不成呢？


Hmm, this looks like something we missed when we fixed the ast cache.


What problem is this trying to solve? The default value of this.state.open is
```javascript
this.state.openImmediately || false
```
This change will cause changes in openImmediately to toggle the Overlay. I do not think the intent was to use openImmediately as a means of toggling the Overlay. Are you trying to open a dialog-window immediately without an Overlay?


这个步骤是一次性的，感觉不用列在这里。而且这是一个开源项目，可以介绍一下用户如何 install，发布流程一般不用放在这里


You don't need the footer and social bar conversion for mobile?


Great feature! Missing from doc/lawrencium.txt, though! :)


We are open to any discussions using currently available communication
tools. In Scala plugin case it's mainly developers forum, bug tracker,
support board, twitter and direct email messages. Additionally we are using
GitHub tracker and comments. So I'm not sure it's good idea to add one more
channel for that, it's already quite time consuming.

Alexander Podkhalyuzin
Scala Plugin Team Leader
JetBrainshttp://www.jetbrains.com
"Develop with pleasure!"

2015-03-12 15:14 GMT+03:00 Dale Wijnand <notifications@github.com>:

> I was thinking more of a way for the community to engage with the
> project/company devs.
>
> On 12 March 2015 at 11:36, Alexander Podkhalyuzin <
> notifications@github.com>
> wrote:
>
> > @dwijnand <https://github.com/dwijnand> Not sure we need that. We can
> > discuss everything here in GitHub or by email.
> > Internally we are using Slack and some own stuff in JetBrains.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/JetBrains/intellij-scala/commit/c2736b9649f711507ae68219b1bc5eef49eeb4bb#commitcomment-10162035
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JetBrains/intellij-scala/commit/c2736b9649f711507ae68219b1bc5eef49eeb4bb#commitcomment-10162568>
> .
>


```
this.defaults.exclPrice = 0;
...
    excl: that.exclPrice,
...
delete that.exclPrice;

```


This may be good to include in 6.0.1, and safe to do so.
@gathreya What do you think?


fixed.


"It doesn't have anything to do with the sorted funtion from python other languages." I wasn't referring to other languages' versions of sorted, I was referring to Scala's `sorted`. http://techie-notebook.blogspot.com/2014/07/difference-between-sorted-sortwith-and.html


added line to meditate


@Calinou will fix with language additions


Sure #327


Logs (except INFO) to be removed.


Please consider it as extremely minor change, but what do you think about using empty lines in code to split logical parts even in such simple methods?

```
    public fun testSimple() {
        myFixture.configureByFile(getTestName(false) + ".kt")

        val source = myFixture.getElementAtCaret().getParentOfType<JetFunction>(false)
        val target = JetQuickDocumentationProvider().getDocumentationElementForLink(myFixture.getPsiManager(), "C", source);

        Assert.assertTrue(target is JetClass)
        Assert.assertEquals("C", (target as JetClass).getName())
    }
```


I didn't about `Math.rint`, nice!
But what if a double can represent perfectly the integral value? For `2^54-1` for instance.
I guess it does not matter too much, since anyway such big values are printed in scientific form with `%g`.


Great work you've been doing here over the last few months! I'm spinning up some new Nagios instances in the next week weeks and I'll be giving this a spin in our environment.


I only wonder why it was merged :/


It depends on how detailed we want to get in describing the system, versus giving the voter a sense of how they'll cast the ballot but leaving the *exact* mechanics of it up to the electoral district to describe to the voter when they get to the booth. For example, does the standard need to specify if a voter MUST cast a vote in each contest? What about the rather extensive and subtle variations on IRV? (That alone could easily triple or quadruple the number of enumerations here.)


oh yeah this should be fixed..


The "BallotSelection" object has already been modified to remove vote counts. Thus the only remaining elements in this object are those necessary to answer the question "who and what is on my ballot?"


bower no longer let you register packages with CamelCase


This and many of the other suggestions seem overly verbose to me, but I'll let others weigh in on this issue.


<!-- 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.


One thought (for later): Kaui doesn't ship with H2 by default. However we could install the Ubuntu package on that image. It's lightweight enough that it shouldn't get in the way, even for production deployments (or introduce an option to skip it).


I've waffled on this one as well, but can see it both ways. CandidateSelection separates the two since there can be more than one candidate for a given contest (e.g., "Obama/Biden" and "Romney/Ryan"). There would be clarity but at the cost of complexity.

I'll let others weigh in on this.


There it is. I can confirm its working with my test streams with another fix for which I'm not sure at all.
The AES encryption for the stream is a proprietary solution where fetching the key must be done with a SSL client certificate and with previous authentication steps with the key server. All this is wrapped in a binary so I wrote a new DataSource that is calling their client instead of using the ExoPlayer AESDataSource. To have the decryption working, I need to force the creation of a new HlsExrtractor: around HlsChunkSource.java:334, I replace

    if (previousTsChunk == null || segment.discontinuity || switchingVariant ||  ...

with

    if (true || previousTsChunk == null || segment.discontinuity || switchingVariant ||

and then it's working otherwise its stops playing at the third segment apparently because the TSExtractor tries to read the segment from some offset instead of from the start so it gets lost when trying to parse the stream. I have strictly no idea if this is linked to my custom DataSource or if it would be the same with a normal AES stream.


If you have better ideas on how to fix it, please let me know.  I was doing the equality check against a casted value to see if it would perfectly fit (within a reasonable amount of precision), but that caused a FindBugs warning so I changed to this.

If it helps any, MRI doesn't track arbitrary precision for this formatter.


Agreed, the linguistics are a bit weird. @jungshadow, @pstenbjorn, and @jktomer: thoughts?


Done ! ;-)


npm install


Mike, I pulled this into my local and it's working exactly as we expected it should.  I did find that I needed to reload my console with every change to the piece.rb file.
from within the console I do the following.
```p=Piece.last
p.is_move_obstructed?(0,7)```

In the console you should see a db query for each row, it will puts the row number and any piece it finds.  Currently, it finds a pawn at row 6, as expected and returns false.  If I change the each to (1..7) it will find a pawn at row 2.  (2..5) finds nothing and returns true.

So, the next step would be to use destination_y and self.y_position for the rows.  I.e. we can start with any piece and go to any destination.  One potential glitch I see is that we can't do (5..2).each in ruby.  So, we will likely need to organize our loop differently.

The next step is to account for horizontal moves and lastly to figure out which direction we are moving and to apply the appropriate logic.


As I already commented in the API project you should not pass the whole config object but only what you need.


Never mind, that was a silly statement as everything is async anyway, sorry for that. I will move it back to its initial place.


It is possible for a single "choice" to have multiple candidates (e.g., President and Vice President grouped together as a single selection). The latest revision (v31) removed the maxOccurs=1 constraints. BTW, I've pinged John Wack from 1622 to see when the next draft revision will be made public; they're still acting on and updating stuff from the latest rounds of public comments.


can you paste the output?


some tests for this would be nice


Again, this feels slightly too verbose to me (and "Ballot Measure" is definitely a thing, just like a "Candidate Contest" is a thing, but "Ballot Measure Contest" feels ... wrong). Thoughts, everyone?


I'm not sure if I understood your third comment regarding re-reading of the configuration file to know that value of ${CVMFS_CACHE_BASE}, and so I don't know if I rightly corrected it in this pull request.. Please let me know..


I saw that switch, but even without it no shortcuts were created. Perhaps this is due to upgrading, and not installing from scratch?


However..
4. Create a method to fetch all courses of a given student.
5. Create a method to fetch all students of a given course.


ㅁ..뭔가 TODO가 많이 붙었지만
환영말 고치는 버전으로 일단 올립..


@jdmgoogle the language was adopted by VSSC and the extension is from `BallotSelection`.  The syntax is a bit stilted because of the desire for all elements to be singular.  If it were called CandidateSelections (which is really what it is) then it would be less awkward.  I see benefit in consistently naming elements that are extensions of other elements so if we would propose a change to `CandidateSelection` it should be accompanied by a recommendation for `BallotSelection`.


For accountability, please always specify a reason. Refs https://github.com/NixOS/nixpkgs/commit/4933a342797c8a18e2794eac28917842987ad71d#commitcomment-10162472


강동훈 (darkpsycho@ajou.ac.kr) 미디어학부08
하고 싶은 것 : 친누나 이기기. 모든 분야에 걸쳐 다 이기고싶음
해야 하는 것 : 설거지, 그리고 프로젝트
할수 있는 것 : 개발!


I'm not crazy about using the reject channel to propagate a successful result.  I'd prefer to find a better way.


This returns a reference to a temporary variable! I think you should use `std::move` here, or just change the return type to return by value.


TPE rulez !


How to edit ?


Thanks. I saw them after. I had 2 diffs and forgot to merge them before committing. :)


I've learned <%= %> tag to use ruby code in html.erb .


oops, I always forgot to remove the generated files! I fixed it.


reset


Looking at this a bit more, I see why you've done it this way.  It may be better if we actually refactor the way `guard()` works in order to make this whole thing easier.


@bchavez :

First of all Many Thanks for your replies on our requests.
And pull the latest again but this time error while building the API solution. Please find the attached.


Sorry, you're right. Really, I did try to double check this to make sure I got it right, I didn't so thanks for catching it! Actually, this has happened to me before in code, for some reason I see "PASS" and think "let it pass (any type errors)".


okay or not ?


+강동훈 (darkpsycho@ajou.ac.kr) 미디어학부08
+하고 싶은 것 : 친누나 이기기. 모든 분야에 걸쳐 다 이기고싶음
+해야 하는 것 : 설거지, 그리고 프로젝트
+할수 있는 것 : 개발!


Not correct


perhaps something like ` return "SOME(" + obj.toStrong + ")"` to make clear that the object is inside an `Optional`?


@masakielastic please forward this to the upstream as well.

Thanks.


oui chez moi ca marche. Mais je compte reprendre  les providers ce week end histoire de réaligner les changements


Okay, we should identify the right place to record this information. What do you think about _input/event.h_?


The build fails here for armv7l. Looks to me like bad style and non-standard. Here is also a typo "iff". Please fix that.

My suggestion is: Default to 0 and put this as a runtime check to early execution in main() or check how other tools detect endianess.
See: http://stackoverflow.com/questions/1001307/detecting-endianness-programmatically-in-a-c-program
and: https://sourceforge.net/p/predef/wiki/Endianness/
````
libtool: compile:  gcc -DHAVE_CONFIG_H -I. -I. -I. -O2 -g -Wall -g -O2 -MT endianness.lo -MD -MP -MF .deps/endianness.Tpo -c endianness.c  -fPIC -DPIC -o .libs/endianness.o
In file included from endianness.c:29:
endianness.h:30: error: initializer element is not constant
make[2]: *** [endianness.lo] Error 1
````
Environment:
Debian Linux + scratchbox for ARM compile.
````
$ uname -m
arm
$ gcc --version
sbox-arm-none-linux-gnueabi-gcc ('cs2009q3-hard-67-sb16') 4.4.1
$ libtool --version
ltmain.sh (GNU libtool) 2.2.6b
$ automake --version
automake (GNU automake) 1.8.5
$ autoconf --version
autoconf (GNU Autoconf) 2.61
````


TEST


hello


This is a really spiffy idea!


Are you using production keys, or sandbox keys?

Are you trying to connect to your sandbox QuickBooks company, or a live QuickBooks company?


Also -- this is not the correct avenue to get support -- please use our forums: http://www.consolibyte.com/forum/


@dzhuang I'm receiving emails from cnsba.elearning.webmaster@gmail.com with error messages from what appears to be your instance of Relate. I'm suspecting that you used the above sample configuration (which I've changed now) which defaults to sending these emails to me.


That rule seems related, but slightly different than what I implemented. Y038 says that if you have a controller associated with a route, you should set the controller in the route definition instead of in the template. What I wrote doesn't enforce that, it only forces you to use controllerAs if you use a controller in your route. If you don't include a controller, the rule I wrote won't display an error.

You could implement Y038 using very similar code to what I did for Y031, though. They're definitely related rules.


I realize the `[]` here means "optional", but may look too much like an array in this particular case.  Not sure what the best way to fix it is, tho.


How about:

"The returned promise will fulfill with the end signal value (if present) after all the events have been consumed."


A naturally place ;-) -  another enum beside TOUCH, something like TOUCH_END ?


dasdasdasdas


dasdsadasdasdada


The word "resolution value" is problematic for promises, because "resolve" means "fulfill or reject".  How about similar to the suggestion above:

"The returned promise will fulfill with the end signal value after all the events have been consumed, ..."


Well, three options:
One, a global just to say "don't use the admin/default account".
Two, a user preference setting that will kind of be hidden and require a join to reference.
Three, add "reminders/messaging" or some such to the actual user's table and put a checkbox in the user UI.
    The third option is the most powerful, and makes the whole mess less messy.  It also provides impetus to take us down the dark dark road of adding other more explicable items into the user's UI for lots of other stuff in pursuit of dumping the hideous GACL (The Gackle Monster, as it is known in some circles).


you can use Ctrl + Shift + F in Eclipse to auto-format the files. For example, to add spaces on either side of the * operator


Je suis vraiment pas convaincu par ce style là


I thought that a course was defined by its term_id, campus_id, and course_id. If we were treating a course truly as a course, I could see a case were a course could be repeated on a campus, but since we are treating course more like a class, then the campus it is offered on is significant.


Would be useful to add documentation about what the function does


I know we can't make a single call to etcd - this is where we typically use
a work pool and to make many calls to etcd simultaneously.

On Wednesday, March 11, 2015, Amit Gupta <notifications@github.com> wrote:

> We're doing creates in etcd, can't batch it.
>
> A test would be great, but it's tricky, we have to either have etcd fail to
> create some but not all of the ActualLRPs, or stub the behaviour of some of
> the intermediate methods between the lines changed in this commit and the
> store client methods, but all the intermediate calls are unexported
> methods.
>
> On Wed, Mar 11, 2015 at 9:57 PM, Onsi Fakhouri <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
> wrote:
>
> > Also: we're still creating the actuals serially we should be submitting
> > batches wherever possible.
> >
> > On Wednesday, March 11, 2015, Michael Fraenkel <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
> > wrote:
> >
> > > Shouldn't we have a test for this?
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/cloudfoundry-incubator/runtime-schema/commit/cb692d996d3a1c659eac5153eeb4e1ed9ac2fc52#commitcomment-10156330
> > >
> >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/cloudfoundry-incubator/runtime-schema/commit/cb692d996d3a1c659eac5153eeb4e1ed9ac2fc52#commitcomment-10157321
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cloudfoundry-incubator/runtime-schema/commit/cb692d996d3a1c659eac5153eeb4e1ed9ac2fc52#commitcomment-10157383>
> .
>


commiting it again for testiing purpose


Etag validation description: http://en.wikipedia.org/wiki/HTTP_ETag#Strong_and_weak_validation


Remove this tr as you want to close table later.


Chrome checked only the active window...
I "fixed" it, however I'm not sure if this will be a bit distracting, since it will focus to another window now, leaving your working window.


Yes, see https://github.com/dimitriv/Ziria/issues/94.


I changed it because validation wasn't working properly


I would not write camel case here since spaceship is a single word.


fdsafdsa


Couldn't we introduce an accessor function ```Input::Event::is_touch_release``` similar to the existing functions ```is_absolute_motion```?


`data_obj[:has_data]` is not available - I defined it myself.  Look at the other files.


Thank you @iGranDav for creating a PR


Looks good! I had half finished it and forgot. Thanks.


Why did you delete your PNG files? They are necessary for the evaluation.


Je pense que dans un premier temps il faut considérer les couches read-only provenant d'urbanmap commme des couches "système" et je mettrais les droits suivants :
- user_commune :
 * u'view_resourcebase', (voir la couche)
 * u'download_resourcebase', (télécharger la couche)
- admin_commune :
 * u'view_resourcebase', (voir la couche)
 * u'download_resourcebase', (télécharger la couche)
 * u'change_resourcebase_metadata', (modifier les data donnée)
 * u'change_layer_style',(modifier les style de cette couche)
Et ceux-ci uniquement pour le superuser admin :
 * u'change_layer_data', (modifier les donnée de cette couche)
 * u'change_resourcebase', u'delete_resourcebase', u'change_resourcebase_permissions', u'publish_resourcebase' (gérer la couche - c'est 4 permitions vont de pair)

Car il ne faut pas (mm un admin communal) retire ces couches, de plus j'hésite même pour le style


Can you be more specific? Output of the install script?

I did successfully install using these changes.


@fnikitin check this with apache


ya, the output of your test run.


@DanielJMaher @oceanzus @lukecampbell

This is an aweful lot of model logic in the application manager, it should be moved out into a controller…or better yet use existing model definitions or expand on them.

Sustainability and reusability!


How does Chrome Settings window behave in such a case? I believe it will focus to another window.. (not in-front of a PC having chrome)
I believe it should work similarly to Chrome Settings window..


I'd say "it goes after the class name (an optional type parameters)"


BTW, protocol-less links (//domain instead of http://domain or https://domain) are better in all the cases I know. When in use, browser will correctly issue secure/unsecure request depending on the current protocol of the page.

So, I presume the only fix here should be -- remove 'http:' part from <link href='http://fonts.googleapis.com...


You can use Ctrl + Shift + F in Eclipse to auto-format the code. For example, there shouldn't be a space before the semi-colon, but there should be one after it.


Sweet! Thanks for fixing the merge issues.


@danawoodman should be fixed in `v.1.1.1`


La manière que je vois ca, c'est que tu fait deja un espace pour déclarer une fonction normal (ta pas la choix), sinon ca brise un peu la logique et ca pourrais laisser croire que ta une fonction qui s'appelle fonction,

Même si on sais très bien que le mot "function" est réservé. C'est juste une bonne habitude a prendre imo.


Are you supposed to be ignoring the `force` parameter here?


This isn't using isolation strategy of THREAD.
Replace timeout property with `hystrix.command.default.execution.isolation.semaphore.maxConcurrentRequests`


Ignore this. drafter now compatible.


timeToTravel is the total travel time, so for each step, the velocity should be multiplied by (timeToTravel / NUMBER_OF_TIME_STEPS) * i


Do we need a comment to explain why the Skin UI's position is important? I certainly don't understand why.


Will be ugly to write each time three lines for a single file in the function `run_all`. It is better to make it through `for`.


Du coup il faudra peut être modifier le status de l'admin communal car pour le moment il est en mode staff et donc a presque tous les droit.


the parentheses (1. + c) might not be necessary syntactically, but i put them there to make it obvious that we want that addition to be performed *before* the subtraction. this prevents numerical cancellation. please leave them in.


по идеи у @admin после создания должен быть метод @admin.confirm! но не принципиально конечно


It is best to use parenthesis when you have multiple operators in the same line:
startPosition + (velocity * (timeToTravel / NUMBER_OF_TIME_STEPS) * i);


Good.


Also, there may be no curly brace, but a primary constructor may still be present


if we use np.sign, we don't need to import sign in the header of the file …


maximumNumber is the maximum value that can be printed, not the maximum number of values to be printed.


Shouldn't it be "**the** init keyword"?


Just verifying after installing on a live shop, the sorting appears to be working as intended.


ditto, log is already imported at the header of the file, so why not use it?


Yes Yes it works. Thank You very much.

On Wed, Mar 11, 2015 at 10:38 PM, Gerd Katzenbeisser <
notifications@github.com> wrote:

> Ad missing controller:
> Did you enable routes for the plugin? This might be the reason for the
> missing controller message.
>
> Ad how to use:
> PayPal redirects the user to the controller mentioned above. Simply add
> the three methods from the readme to your AppModel. You can simply just add
> your code for completing payment in afterPayPalPaymentExecution() and
> ignore the others.
>
> Actually beforePayPalPaymentExecution() is here to check some
> preconditions for accepting the payment. For example guess an order item
> has expired and you don't want paypal to actually perform the payment you
> can return false here and the payment will not be completed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hakito/CakePHP-PayPalRest-Plugin/commit/2d58063066907e8e1a7a5455b8c1f6427582c9a9#commitcomment-10152802>
> .
>


Chrome opens an Options page for each window.


It can not be equivalent to the previous code, because the class name is different and there're three properties instead of two


Ah, that's what they are for! Ideally, take over my commit, put them back and add a commit saying just what you said now --- for the sake of whoever next tries to 'simplify' the expression.


@moul Merge this to develop for now?


"Declare" would be better, instead of "contains"


Why does History Graph show much lower speeds than the individual dial readings?


Should https://exercises-dev1.openstax.org/api/ be moved to settings.js?


hell yeah


oui je pense, il pourra avoir tout les droits sur ces couches (uploadées à la mano) mais pas les droits de casser urbanmap (enfin dans un premier temps)


Ah, pretty good place - I did not catch it because I only read the diffs here :blush:


"to another constructor of the same class"


We are trying to move away from `numpy import everything` style to `np.something`. Not a big deal, I wouldn't worry about these too much were it not for adding it to the list in `__init__.py` (this way these new distributions automatically end up in the html docs)


iff means 'if and only if'


What about calling `super`?


super


should this be translatable?


Yes, g++ will be automatically installed


I think the variables naming style is different, please double check


True! Thanks.


i tried to de-emphasize the scale issue by putting parentheses, because technically, if we say that gennorm with beta=2 is identical to norm scale=2**-.5, this is only true if we also set gennorm's scale=1, not for any value of it. but saying it that way would be quite tedious.

besides, "a normal distribution" refers to any normal distribution, not just a standard normal distribution, so it is not wrong to omit making a statement about the scale altogether. although i agree it is useful to alert the reader to the scaling issue, so they don't jump into false conclusions.


Bah, commit message. This allows for empty brush overrides :\


Adding this accessor is ok - but I would still need to use/propagate the rx/ry values as end of touch events, right ?


For the moment, I'd change the testfiles to match the actual output of
pandoc-citeproc which I believe to be correct now for BL @book (@inbook was
never affected).
Actually, the issues with title-short we're seeing now could be easily
solved if CSL used a new multivolume-title (or so) variable, and BL
title/maintitle could be mapped to CSL title/multivolume-title; I've
proposed this at the xbiblio mailing list and
https://github.com/citation-style-language/schema/issues/111.

On 8 March 2015 at 22:22, John MacFarlane <notifications@github.com> wrote:

> The title-short changes I made were needed to get the test suite
> to pass, so probably if they are wrong, it is a problem with one
> of the changes to the code that you made. Can you see what needs
> to be changed back?
>
> +++ nickbart1980 [Mar 08 15 14:53 ]:
> >There's nothing wrong with "title-short: Title of the “@inbook” Entry" in
> inbook-title-booktitle-maintitle-series.biblatex and
> inbook-title-booktitle-maintitle-series-2.biblatex, and "title-short: Vom
> Nutzen und Nachtheil der Historie" in nietzsche-historie.biblatex.
> >(For @inbook etc., BL title maps to CSL title, so mapping BL shorttitle
> to CSL title-short is fine. @book is more difficult: There is no BL
> shortmaintitle, but maintitle (without mainsubtitle/maintitleaddon) could
> be mapped to CSL title-short for @book etc. containing maintitle; I didn't
> figure out how to do this, though.)
> >
> >"title-short: TeX" in knuth-ct-b.biblatex and "title-short: METAFONT" in
> knuth-ct-d.biblatex are incorrect and should be deleted: these are not
> short forms of the title.
> >
> >BTW, while working on this, I realized that the idea of using
> volume-title in CSL, brought up by one of the CSL folks IIRC, is itself
> problematic, less so for CSL chapter, but for CSL book, where title is no
> longer title of the smallest/most specific unit, volume-title cannot easily
> be used for sorting etc. The biblatex approach, where title is always title
> of the smallest/most specific unit, appears to make much more sense. Most
> likely, his will have to be discussed with the CSL folks again …
> >
> >---
> >Reply to this email directly or view it on GitHub:
> >
> https://github.com/jgm/pandoc-citeproc/commit/5a62f7480e37a11fa698797d5fa601380bda84da#commitcomment-10091108
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jgm/pandoc-citeproc/commit/5a62f7480e37a11fa698797d5fa601380bda84da#commitcomment-10091220>
> .
>


A link would be nice here for "package-level functions"


Ahh, I see it. Then our fix works similarly right?


Thanks. :smile:


object declaration (no "s' at the end)


However the multiple Chromium _Settings_ tabs are all synced in real time as we change settings. Not sure I want to go that far with uBlock = adding complicated code for such a minimal gain.


Right, I do think that an automake test would be better.


@Tommy-Geenexus Should be fine anyway :)


please remove the word "if" and specify the reasons  of the if and else parts


I'd refrain from using the word "static" here without a qualification that it is an analogy to a specific language (Java/C#/C++).


Link to bug ticket?


Strictly speaking, this is an obsolete statement, but the observable effect is the same, so we can keep it, I think


a class or trait


Ah okay, thanks! But the build issue still has to be fixed. I was surprised that this commit got merged so fast.


Ok. No Problem at all.


`-u` might do more than you want:

```
Search all files. This ignores .agignore, .gitignore, etc. It searches binary and hidden files as well.
```

It ignores your `.gitignore` and searches binary files. `--hidden` just finds files that start with a dot.


This is awesome!


@ahilles107 why did you remove it ? api exceptions are not in json format now and Newscoop exceptions are in json.


how could i get each instructions cycle count?
by data sheet  or  debug(i don't know how to use gdb in qemu)?


@rbf : Ok for the french version.


Infinite recursion will eventually cause SIGSEGV segmentation fault. Your "perfect" bot is not perfect. See pull request #1.


Fixes https://github.com/ESGF/esgf-installer/issues/45


@ralgara
I have added test case of https://github.com/caolan/async/pull/512, but it was already corrected.
I would appreciate if you could check it.


Outdated by e0edc5efe8342740332a0c2b4b564b0f285fb2a1


why not make this cast on instantiation, instead of every time we ask for the size?


Isso mesmo @RummeniggeMaia! Isso ajuda na hora de fazer merge!
Cada desenvolvedor em suas máquinas usam tamanho de tabs diferentes!


Foram adicionados alguns comentários para fins de documentação (javadoc)


s/priodiy/priority

Also this means that using both at the same time is ok. Are you sure that's true and that they use the exact same versions? Shouldn't we use javax.annotation.annotation-api instead and remove jsr250-api from our dependencies? If they're the same, there's no reason this shouldn't work!


parenthesis: OK, it's a shade of the color for the bikeshed. I removed parethesis simply because my eyes staggered on )). Either way is fine by me.

scale: I understand your point about "a normal distribution" vs "the standard normal distribution". Still, we have quite a bit of confusion about the whole location/scaling, and there are at least three issues/PRs flying around right now (https://github.com/scipy/scipy/pull/4611, https://github.com/scipy/scipy/pull/4559 and references therein). Being specific here does not hurt and IMO makes sense. As long as we are not going overboard and littering every other line with exact tedious details, I agree :-).


pull master in sync with gh-pages (how can I avoid that??)


My mistake. I originally referenced it, but apparently deleted the check while refactoring. I'll readd it when I get home.


With pleasure :-)


Please add a service name or 2.


Yes, but I would keep this as an implementation detail internal to the ```Event``` class. I guess your question refers to the arguments to the ```Event``` constructor? Maybe, we could introduce a statiic function ```create_touch_event(ax, ay, id, touch)``` that returns an ```Event``` object but has no rx/ry arguments? Internally this function could pass the magic -1/-1 arguments to the ```Event constructor``` it ```touch``` is false.


GraphicsEnvironment.isHeadless();

You would need to do this to see if a GUI is supported


typo on line 8:
'require reqeust'


@aleos89 your welcome :+1:


@tomlee1968

I've replaced ssh with autossh in sshtunnelstart() so that it will automatically reconnect if the connection closes. However, I am not sure how to handle the sshtunnelstop(). It's using ssh -O exit, and I am not sure how to do the same with autossh. I think it's easier to just kill autossh process? Anyway, please update sshtunnelstop().


"take over my commit": See Pauli's comment in https://github.com/scipy/scipy/pull/4500 for a git way with `git remote add` or just do it manually under your name. Either way is fine by me.


adding comments...fixing test2.


@jforberg How does the cancancan authority-checks look in the views?


of course, it's a WIP


what would you mean?


Berufsbildungszentrum Solothurn-Grenchen


you're reading my mind! i was just trying to figure out how to do that :))


Андрей, очень нужны комментарии как теперь это работает.

К примеру было выяснено что

1. Картинки ужимались по максимальной высоте и ширине на основании параметра

$config['module']['image']['preset']['default'] = array(
    'driver' => 'Imagick,GD', // 'GD', 'Imagick' or 'Gmagick', or several libs separated by comma

.........

    'size' => array(
        'width' => 750,
        'height' => 750,
    ),
);

Здесь я вижу что height -  убран а width остается. Почему?

2. Означает ли что теперь именно в админке задается максимальная ширина? Только ширина?

3. Что означает измнение в шаблоне? Там будут меняться проценты по какому-то закону?

4. Это значение - максимальная ширина, как выяснили еще привзяано к размеру фрейма при вставке видео. Как с ним?

Честно говоря, когда я постил эту проблему я не думал что тут так все лихо наврочено и сейчас предпочел бы чтобы это осталось как есть, т.е. допустим параметр задается в конфиге а в админке вообще бы эту возможность убрали, или просто - продублировали, так что параметр в админке был бы эквивалентен этим значениям ширины и высоты в конфиге. Очень боюсь что сейчас после этого обновления снова вся логика с вывовдом картинок порушится.


new fixes for readme...


Not particulary happy with this. It's the only place I can see a `NancyContext` from inside `NancyMiddleware` without modifying `Request` and having a knock on affect from there.

On the other hand, I'm leaning towards the face that `User` should be a request property and not a context property with would work better here. But that would have a lot of breaking changes too...


i am lost with this toool....


do we need to add a website field? - keep in mind this is just a template, so the simpler the better.  Makes sense that you were testing out how to add things


It's fine to leave it, as I don't think it hurts anything.


Also tried to fix #254 and seems to work


Thanks.


may be better to use mb_strtolower
it is actual for names not in IDN format


Maybe we can add the siteNodePath as example ?


i.e as you can see here: https://github.com/holylobster/nuntius-android/blob/master/app/src/main/java/org/holylobster/nuntius/Connection.java#L40 we do not prepend to the variable names 'm' i.e that variable would be called recyclerView


I would suggest using a more descriptive name like workflow.


Just a reminder: This variable should be modified too =) https://github.com/prismicio/python-kit/blob/master/prismic/__init__.py#L6


:+1:

Next step a global dataProviderOptions merged with the local one to avoid some configuration duplication, or can we use YAML reference ?


go go go


@lewicki I have done a feature branch on dexterity implementation, called dexterity-behavior, may be of some use for you.


could you organize the imports according to https://www.python.org/dev/peps/pep-0008/#imports ?

So this would end up
```python

import os
import re
import subprocess

import diamond.collector
from diamond.collector import str_to_bool
```


ok.


$ git config --global user.name "your name"
$ git config --global user.email "your_email@youremail.com"


we already use a global config and reference to a 'databaseSource', I could move that to this package too


Looks like StratusLabIterClientCloud is not imported.


/gino


True. Will change that.


gino @Fupete


@federicoloporto prova


looks ok to me


@nunziap90 prova2


done


6: M102 unrecognized bullet/signature


@sloria - this prevents JSHint complaining about using `window`, `document`, and similar objects.

See http://jshint.com/docs/options/#browser


Can we add this check to the ErrorHandler so it can propagate a `ResourceNotFoundException` instead of an `HttpresponseException`?


Please properly escape the string translation result before printing it.


:+1: Nice!


I should have read the `disable_icanon()` before removing it. I guess I was too much in hurry. Thanks for the feedback.


Same concern as above.  Needs escaping.


This is a basic version of what should be done. It doesn't address the case of quoted arguments that are passed to gdal_translate and you should requote. For example if someone calls the gdal_translate binary with gdal_translate "in with space.tif" out.tif, the quotes around "in with space" will be removed by the operating system/shell and you would have to put them back. This routine shall also be factored so that all utilities can call it (and the truncation case should result in an error). And for some utilities 1000 character is not enough (for example if using gdalbuildvrt with many input files), so dynamic memory allocation would probably be better.

Actually the approach of passing the string of arguments to the GDALTranslate() function is something that might be reconsidered. There was a discussion about this initiated by Robert Coup (@rcoup) , another GDAL committer who might mentor/co-mentor,
http://lists.osgeo.org/pipermail/gdal-dev/2015-February/041107.html
Maybe this is something you could try to discuss on the gdal-dev mailing list to get more feedback. The pros and cons of the approaches should be weighted in. I can see Robert's point. One point that isn't easily addressed with the pre-parsed argument approach he suggests is how to deal with new options in the utilities without breaking the API. Another point is that some utilities have litteraly dozains of options (see ogr2ogr) and you don't want a function with 30 parameters (10 is really a maximum). Maybe an intermediate approach could be taken, with a basic set of arguments "hard-coded", and for the rest of them a char** papszOptions approach.


you could also call: "make test ARGS=--output-on-failure". I have no preference, though.


You need `var` here, otherwise `field` is undeclared and made a global variable.


@luke E.g. this.


> they contain an empty OP_RETURN output

Those are mine. I'm tracking network adoption for quite a while by sending two transactions from the faucet - one with OP_RETURN, one without.

Imho all this should be specified first, at least loosely, in particular I'm interested in knowing what's going to happen with obfuscation and the marker, and when which encoding scheme is used.


Good point, I released 1.2.1 with the version number correct in both files.


What was the rationale for not adding Command to the class names for the same named modules, e.g.:
module `slipstream_stratuslab .StratusLabTerminateInstancesCommand` and class `StratusLabTerminateInstances`?


Good point ;-)
I will fix that.


This is the documentation for getMu() not for more than one method, so you shouldn't document more than one method here.


Is this test really necessary ? The only reason I see for this test is that when we will build with clang, we would have a useless repository.


What makes you think that it is not clear why it was added? Sure, the comment could be better.  I added the hack back then exactly because of errors like you have seen, only the other way around. Basically you have the xml stuff in the JDK and you have a library that brings the xml classes as well. If both parsers are used, you get into problems, because they use global states. That's why it is in general advised to add those xml classes containing jars into the endorsed directory of your JDK.

So while the removal here will solve your problem, other will have the problem again. I see two possible ways to fix the situation. One is a System property to switch on and off this behavior.  The other is to not to check for "org.w3c.dom.Node", but delegate all class loading requests starting with "org.w3c.dom." to super. And actually both could be combined... Meaning, to load "org.w3c.dom." clases always through super (like a normal classloader should do) and a property to switch that off. I am almost sure, that if we pass through every org.w3c.dom class, then your case will be fine as well.

What do you think?


The problem is we need to know the set of required args in order to be lazy enough with the `extraArgs` definition below. If we know which attrs the called function expects, we can define a set which contains them. If we don't, though, then we just have to pass in `config.__internal.args`, which nix will then try to force to get the attrnames when the function is called, which will cause an infinite recursion because we can't access `config.__internal.args` in that module.

The only alternate solution I can think of is to change nix so that `{ foo, bar, ... }: <exp>` is syntax sugar for something like:
```nix
_internal_args_: let
  _internal_argcheck_ = if _internal_args_ ? foo && _internal_args_ ? bar then null else abort "expected argument whatever";
   foo = seq _internal_argcheck_ _internal_args_.foo;
   bar = seq _internal_argcheck_ _internal_args_.bar;
in <exp>
```

That would delay errors where the function args don't match (extra args for non-`...` functions, args not a set, missing arg, etc.), but would fix this case.


@goeddea we probably want to add *.pid to the .gitignore, since we probably don't want to check this in


While you're here, you might as well either make it look like "num_disks-1" or "num_disks - 1".  Either is fine, though the latter style we use earlier in the line too.


fixed in latest commit.


Why is this migration file in the PR ? Could you sqash your commits to only touched what's necessary ?  Thanks


same question regarding the test. here, the only drawback of removing the test is that with clang, we would have useless packages. But I'm not sure that's a big issue. On the other hand, removing the test will simplify the maintenance of this file


Please elaborate in the commit message why you need a more recent cmake version + see inline comments


teste


Could you please copy the contentd of you config.yml and worlds.yml to gist.github.com, create secret gists, and paste the link to it here?


Hi Roman. Thanks for the encouragement :) Three things:
* It's still buggy, verrrry buggy!
* I'm thinking of offering the option with/without artwork. Artwork can kill a small device, I think.
* It needs some documentation.


Usually I'd expect a saveStatus(IvFluidOrderStatus) method instead of setStatus. (In fact OpenMRS will automatically handle some things in this case, e.g. automatically setting creator, dateCreated, etc.


`StratusLabTerminateInstances` is because I kept the original name that was in `stratuslab-terminate-instances` and I added `Command` in the file name to be more consistent with the file `StratusLabCommand`.


there is a file 'ccf-format.txt' in this same directory.  this looks like a copy of that.


add the controller and the model there is nothing called db server in the uml and edit the user branch as it should be the same as the view and the controller


It has db changes. So if we do that well have to do oracle scripts for it too. I am ok with that if this is important


Hi there!


Hi,
I have tried your code for sonar. Everything was ok. But I have some questions. Why is there 1 kOhm resistor. My intuition is that i need a voltage divider. There shoud be  5V, so I need somethig like 2kOhm and 1kOhm resistors to get 2/3 of 5V, accordingly 3.33V. You used just one resistor. Whats the explanation? Is there something like inner resistance of sonar?

Thanks

DD


@return ?


One possibility that I forgot to point out is that you could do what we do here:
    HibernateScheduledDoseDAO extends SingleClassHibernateDAO<ScheduledDose>


Actually - I'll handle this.  Don't worry about it.


typo: should be ebola_ivfluid_order_status


ObjectGuild  byte variables in length


Great, let me give it a shot, will update you soon.

--
Sent from my iPhone

> On Mar 12, 2015, at 8:32 AM, Bogdan Andrei IANCU <notifications@github.com> wrote:
>
> Finally, this was fixed, see the f612548 commit or simply pull the latest trunk.
>
> —
> Reply to this email directly or view it on GitHub.
>


Default is 0, right?


the following three lines can be replaced by
```c
LED[c]=(k>>c)&0x1;
```


ATTENTION :warning: Tu fais un bouclage de données ! la class Map contient les monstres et les monstres ont la Map en paramètre...


Great =)


I'm not sure why you changed this, readability? It doesn't have anything to do with the leak. By the way I realised recently that folding a shift greater than the bitwidth for the arithmetic right shift is completely wrong! If it were correct then the following SMT-LIBv2 query would be unsatisfiable

```
(declare-fun x () (_ BitVec 32))
(declare-fun y () (_ BitVec 32))
(declare-fun result () (_ BitVec 32))
(assert (bvuge y (_ bv32 32)))
(assert (= result (bvashr x y)))
(assert (distinct (_ bv0 32) result))
(check-sat)
(get-model)
sat
(model
  (define-fun y () (_ BitVec 32)
    #x00000020)
  (define-fun x () (_ BitVec 32)
    #x00000000)
  (define-fun result () (_ BitVec 32)
    #x00000000)
)
```

It is correct for logical right shift and left shift though.


The clear_cache command used by update.py is https://github.com/mozilla/treeherder-service/blob/master/treeherder/model/management/commands/clear_cache.py
We can leave django_extensions in dev.txt imho


dont use standard libraries for such a small baremetal project


hahaha, sensacional cara :) Acabei nem continuando o projeto, but nice! Vou dar uma olhada nas changes e ver como posso contribuir.


The 'GrabFromAtlas' command isn't perfect, but it gets the job done.


What happens if I send a "q" from the PC? :bulb:


do something more interesting in main to filter all other xters from PC before displaying, we said only display [0-9]


- `self.to_handle` is unused
- `self.rpm_ts` is only used in `_src_deps()` so there's no reason to define it `__init__()` and not just localy in `_src_deps()`.


I'd rephrase this the following way:
`An integer representing the delay (in milliseconds) before snapping to a panel.`


move the alt box as before in both cases it goes back to the controller


yeru


I had to change lines 18-19 from $INSTALLDIR to $DIR to be able to build fc21 template modules.

I do not know yet if that will effect the building of the template or if it breaks fc20 since I just ran into the issue.

I am off to bed for a bit and can take a closer look at it later today if you have not already fixed it in your repo.


Nice commit comment! ;-)


results.passedExpectations is always true (it can be an empty array though).
Should be
if(!(self.takeScreenShotsOnlyForFailedSpecs && results.status === 'passed')) {


Fix the order, should be below `slideSpeed` like in the rest of your PR.


this really isn't needed in the CI context.


Sounds like we'd need an additional prop to provide classes for it. Pull requests always welcome!


Just


Don't remove it !


Wrong commit title ...


Trying to remove from my view.


Maybe also set this to unique="true"?


This should also be required.


source repos are not needed for .spec or src.rpm; it just takes time to parse them for no use


Уважаемый стажёр! Извольте, пожалуйста, писать все стили в специальном файле, если уж создали его, а так же называть айди и классы менее абстрактно. Исправить!


サムネイルのURL形式が変更されたら終了な機能。


Название текущего файла тоже приболело


use ```$defined_hashes = \Config::get("asset_hash", []);``` to make sure there's an array there.


resultat => résultat


color


I think there's a simple way to amespace the asset_hash into js config group


Looks pretty standard. utACK


test


-- Module Name:    led - Behavioral

library IEEE;
use IEEE.STD_LOGIC_1164.ALL;
use IEEE.NUMERIC_STD.ALL;

entity led is
    Port ( Clk 	 : in  std_logic;
			  Reset	 : in  std_logic;
           Led_out : out std_logic);
end led;

architecture Behavioral of led is
	signal counter	:	unsigned(24 downto 0);
	signal selector:	std_logic;

begin

	freq_divider: process (Clk, Reset)

	begin
		if Reset = '1' then
			counter <= (others => '0');
			selector <= '0';

		elsif rising_edge(Clk) then
			counter <= counter + 1;

			if counter = "1000000000000000000000000" then

				selector <= not selector;	-- Frequency selector

			end if;
		end if;

	end process;

	-- Frequency Selection assignments
	Led_out <= std_logic(counter(21)) when selector = '0' else -- 0.35seconds duration
				  std_logic(counter(23)); -- 1.4seconds duration

end Behavioral;


probably should be

``` return empty($defined_hashes[$filepath]) ? $defined_hashes[$filepath] : '';```


Whoo, congrats! When is the defense? ;)


Looks good.


Ah, there goes my question. :smiley:


Are you sure you need to rebuild from scratch ? Instead, it may be enough to run a cleaning target (rebuild_cache ?) or you may have nothing to do but run "cmake -DCMAKE_BUILD_TYPE=Debug .." ?


HumidifierSizing_#4688 (Nigusse) - 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)


Good


I think we only need to forbid *in the case of* autoref in method form, not unilaterally. That's easy enough to do as we already do some "fixup" in that case.


Nice! Thanks, @wpjunior


В UI принято начинать что-то делать на отпускание кнопки мышки. Можешь проверить. Наводишь мышку, нажимаешь кнопку, кнопка нажимается, но ничего не происходит, и только после отпускания кнопки начинает, что-то происходить.


add size field


I have this installed on a live shop and all appears good.  Allowing the shop owner to pick their own separator has the benefit of using text rather than pipe, comma or whatever.  I am trying the words "Available At" so the Manu title is something like  Super Great Brand Available At My Shop Name.


:P


I think you're missing `)` at the end of the line.


如果对吞吐量有要求， 加上 pack 可以减小包的体积。


"PYTHON_BINDINGS" instead ? (change in commit message too)


no feature guard here ?


Something that might be worth noting on this release - my generated class names changed. They are better now, but just to note it down.

My class name for file something.schema.json was: `Something_schema` and is now: `Something`.


I'm OK with that, if we follow the same approach in all other *Command cases as well.


Старайся стразу использовать переменные вместо захаркоженых констант. У меня на экране iPhone'а буковки малюсенькие и сплющенные. На iPad'е тем более.
Заведи переменную, например: MenuTextHight, ScreenHeight, ScreenWidth и используй их. Если так не сделать, полировка программы под другие разрешение может превратиться в ад.


Hey,
for the first test run I'll stick to commit=35, it's already set up and running. Once it's done I'll download the new version.


I think this refactoring and the similar ones below actually reduce readability both logically and syntactically, at least for me...


Ok


Yes, it should be ==. My bad.


there is a different lenght when the GUID is of creature or of player

ServerToClient: SMSG_CANCEL_AUTO_REPEAT (0x1959) Length: 9 ConnIdx: 1 Time: 03/10/2015 02:21:44.875 Number: 9652
Guid: Full: 0x080270000000000000000000085056EF Player/0 R156/S0 Map: 0 Low: 139482863

ServerToClient: SMSG_CANCEL_AUTO_REPEAT (0x1959) Length: 14 ConnIdx: 1 Time: 03/10/2015 02:21:49.187 Number: 9789
Guid: Full: 0x1C01246B8038844000002D00007E5490 Creature/0 R73/S45 Map: 860 Entry: 57873 Low: 8279184


Можно и здесь сразу переключиться на другой экран. Не надо усложнять код без неообходимости.


Interesting, I was not aware of autossh.  But I think there may be no need to change sshtunnelstop.  It seems that sshtunnelpid works just as before -- it uses the control socket to request the tunnel's status with the 'check' command.  So sshtunnelstop should be able to use the control socket to tell the tunnel to exit normally with the 'exit' command.  And since the tunnel ssh process exits normally, autossh shouldn't restart it.  Of course I could always test it.

And no, autossh goes right ahead and restarts it, so we'll have to change it.  Well, in that case we'll have to do something to remember the autossh PID when it starts, so we can find and stop it.  ... and autossh has no command-line option to write its PID to a file or anything.  Figures.  OK, we have the regular ssh process's PID; we could always use ps to ask it what its parent is.  (Using 'ps' to search for instances of autossh is not a good way to do it; just because there's only one autossh process now, that doesn't mean there will never be others.)  This seems to work.  I pushed this change to the master but haven't put it in place in /usr/local/pf-socks5 yet.

I have tried to simulate a situation in which the tunnel goes down by doing a 'kill -TERM' on the ssh process. Unfortunately autossh seems to take this as a normal exit, so autossh exits too.  I also tried a 'kill -KILL' ('kill -9') on the ssh process, but this also causes autossh to exit, and what's more, it leaves redsocks running, so the initscript doesn't really know what to do.  I'm not sure how to simulate the closing of the tunnel.

Now for what this is all for: I thought the problem was that redsocks kept going down, not that the ssh connection was disconnecting.  Autossh won't do anything helpful if redsocks goes down.  Or am I not understanding the problem?  Was it the ssh tunnel that was disconnecting?


mark here.


Thanks for mentioning this. I expect this relates to the PR that introduced
many changes to the way input files are read.

Could you describe a little more about how the files are specified and what
the context of this generated name is? It's a little worrying that this has
changed so it would be good to narrow down the exact details here so we can
decide if anything needs to be fixed.
 On 12 Mar 2015 14:39, "Sam Duke" <notifications@github.com> wrote:

> Something that might be worth noting on this release - my generated class
> names changed. They are better now, but just to note it down.
>
> My class name for file something.schema.json was: Something_schema and is
> now: Something.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/joelittlejohn/jsonschema2pojo/commit/22da114c07894cfc9c14be821ffa208c157c34a6#commitcomment-10165201>
> .
>


@ogrisel We might want to cherry pick this commit in the 0.16 release branch.


@ryanhamshire Hey, is there any jenkins server available (developer builds)? I would love to check the new features before they hit release, and so provide a feedback on bugs. Great work btw.


in that case we can wait.  thanks


incorrect parameter name


Hi Anthony,

I wanted to ask if you could add the following to the prop header (this
will be important for my next update)

SOURCE : POINT/WALL/Z2WALL ....
BASIS : CHIRAL/NREL
PRECISION : SINGLE/DOUBLE
ENDIAN : BIG/LITTLE

I plan on wrapping these into the "propagator" type and getting all the
prop info from the header. We won't have to worry about implicitly
putting this in by hand in the input file and is a lot cleaner in
general. Before we run too many jobs I would like this done as the newer
code is going to expect this.

Jamie

On 15-03-12 10:41 AM, afrancis-heplat wrote:
>
> Hey,
> for the first test run I'll stick to commit=35, it's already set up
> and running. Once it's done I'll download the new version.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RJHudspith/Contractual_Obligations/commit/d1b0a16e1af9936877c4321380a7f3e62f960f08#commitcomment-10165230>.
>


why didn't you leave the old function as deprecated?


sortWith() method accepting a Sorter added. SortingRequest class now no longer needs to be depended on if clients wish to use the new constructor in SortingRequest.


Or consider another one. Type it in the browser console:

```javascript
function ro() {
  "use strict";
  let x = (function oh_my_this_is_bad() {
    debugger;
    return x;
  })();
  return x;
}
```

Then run: `ro()`.

The debugger will stop you right before x is used, and you can try to access x:

```
x
> Uncaught ReferenceError: x is not defined
```

But then if you do `x = 10` and continue, there will not be any errors thrown after all!

Clearly this is a runtime error, not compile time. We did not change the code, only modified the data in debugger -- yet the error went away.


repositOry


We should also say that you have to choose an available site for the social application:
![django-allauth-chosen-site](https://cloud.githubusercontent.com/assets/71928/6620076/a4050c7c-c89c-11e4-9b0a-32cf56da6e04.png)
This has tripped me up multiple times setting up django-allauth social applications for my local dev environments.
Maybe easiest to say "Click 'Choose all' in the 'Sites' section" or something like that.


leave old function as deprecated?


so now it's rpm-master and not f20-master?


Should be <h4>?


Still f20-master mentioned here.


Okay, I modified the code so that we don't require c++14 features. I'm not really satisfied with the solution, but at least it works. I'll go and work on something that is easier to use.

Here is the modified code: https://gist.github.com/zoq/dcce503eb06d92b6cd42


:+1: :runner:


ServerToClient: SMSG_SELL_RESPONSE (0x0933) Length: 26 ConnIdx: 1 Time: 03/10/2015 02:27:13.703 Number: 23033
VendorGUID: Full: 0x1C01246B8035A7C000002D00007707D9 Creature/0 R73/S45 Map: 860 Entry: 54943 Low: 7800793
ItemGUID: Full: 0x0C01240000000000400000063614A18A Item/0 R73/S4194304 Map: 0 Low: 26677125514
Reason: Ok (0)


(No action required) This looks like something people may be interested in automating...


small typo: repository

+ non-blocking review


你这样改是错的啊
更新的在这里
https://github.com/xiaoliwang/app_missevan_sso/blob/master/controller/sso/update.js#L73


The whole procedure makes sense so :+1:


done


Not suggesting that you do that, but for my own learning could this be changed to `{ _ >>= _ }`?


removing the python guard isn't related to the rest of the commit


Please don't rename this class, this breaks the API


@Dierk, you seem to have forgotten about the link URL itself.


I've not patches delorean and changed all repos, so this is forward looking.
I need to come up with a working migration plan.


Must explicitly supply a bootstrapper instance - a safer and saner thing to do wrt OWIN pipelines.


You are using "FALSE" as default value here but use "ON" at other places. According to cmake manual, option() supports ON/OFF as possible default values.


Fine for me.
I won't close the related JIRA, because we have to test also all Controller definitions. If we detect the use of the $scope object inside the 'controller' definition, we will throw an ESLint error, explaining that we should use the 'this' syntax instead of the scope.
Are you OK with that ?


@joelittlejohn My set up is pretty standard i believe. i am using the gradle plugin, but not using the new android compatibility part ( i got it set up before that fix was put in). my files are specified like so:
```
source = files("${project.rootDir}/../server/schema")
```

and are all of the form: blahblah.schema.json

The `.schema` part was clearly being considered part of the filename previously and now is not.

I'm not sure what I would consider the right behaviour to be. I suppose there are cases where you might include a `.` in the filename and for the components after that not to appear in the classname would probably be an error.


Is it possible to generalize this by allowing `#` in *all* infix operators, and rejecting them in the typechecker?

Maybe not for 4.02.2, but keep this idea in mind for 4.03.


Why this isn't a case class, I note case classes are my default in most cases, just to save myself the `equals\hashCode\copy` hassle


BUILD_TEST (cf commit message) or BUILD_TESTING ?


It's syntax check. In would be added in Ukrainian translation soon.


HumidifierSizing_#4688 (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1644 of 1656 tests passed)

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


a


Is this a JVM bug?


What does it fix? No test?


Fixes #23.


True, I will remove it


Do a case sensitive replace to make sure that doc comment also makes sense? Also here you need a special case for Multistream.


@ericcf does that mean, that whenever we tag an app or an engine we should create a pull request, but we can merge it ourselves?  Thanks!


ok


J'ai le même problème, les fichiers téléchargé font 0k et sont considérés comme invalide.


Yeah I should have added a test. Will fix.


głupek:D


I lost it during a git push -f :(


It's in the LOCATION header of the SSDP packet.


That'll be in 10 hours.


yes. LED and button are used for application purpose. If I/O peripherals are to move to the top level need to have one more bridge in the PCIe sub-system  for BAR 1.
By removing the clock-crossing bridge for BAR 0 in the PCIe subsystem and exporting  BAR 0 to the top-level  qsys directly, then how the clock and reset to be handle for BAR 0 in the top-level qsys?
I feel like we cannot export BAR 0 directly to the top-level (BARs are operating with the pcie-core clock and we cannot export the pcie-core clock to top-level qsys),clock-crossing bridge/pipeline bridge is must needed.


Farbe auswählen für die Suchergebnisse


唉，也无所谓了，改成 update 也可以，就是这个验证的怎么删掉了 Math.floor(sess.loginAt.valueOf() / 1000) === vt.timestamp


ya...


First half of issue #3


I do not agree, building in the sources is a bad idea, it can hide errors. The official position is to always build out of source.


True, I will change that. + it will test the incremental build.


why you doing this?


same here


and here


HumidifierSizing_#4688 (Nigusse) - x86_64-MacOS-10.9-clang: OK (1638 of 1650 tests passed)

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


I think I agree with this after a quick look but will have a more careful read before accepting, hope that's OK :)


@hpalikareva , you had a comment justifying the commented out code ("// Design decision: ..."), which seems to suggest that you were avoiding a double-free error, so perhaps you might like to double-check this?


This will prevent to be able to extend the version finder has the comment was saying


Is this correct?


Of course :)  Thanks!

On 3/12/15 9:03 AM, Dealga McArdle wrote:
>
> I think I agree with this after a quick look but will have a more
> careful read before accepting, hope that's OK :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/shulmang/sverchok/commit/1d87e0a28a2456cb430c4e93e638d0db4f9a9fad#commitcomment-10165970>.
>


you have been known for using long option names ;) (--directory-prefix=prefix)


Thanks. If you point me to the other filters, I'm happy to at least start on the documentation for them if you want. I pushed up changes based on what you said above.


> The clear_cache command used by update.py is https://github.com/mozilla/treeherder-service/blob/master/treeherder/model/management/commands/clear_cache.py
> We can leave django_extensions in dev.txt imho

Ah yes you are right :-)
I've removed this commit and then added one at the end that moves 'six' from checked-in.txt to dev.txt, since now the only things using it are in dev.txt, so it doesn't need to be accessible from production.


I think my brain just broke


I think the reason I had `get_matching_awcs` in here was to include awcs which don't have any cases.  Such a thing might not exist, in which case that'd be a moot point.  Still, since this is now filtering by user_id, this should be sure to include a row for every user matching the filter set, even if they don't have any cases.  Or do you think this report should exclude such data? (the BPR, for example, excludes cases which don't require payment)


LGTM :trophy: :trophy: :santa:


Extract to instance method?


Bonjour Sarakha63,
Merci pour ton travail.
Est-il possible d'ajouter des logs (debug par ex.) dans ces scripts ?
J'ai plusieurs releases qui ne sont pas trouvées alors qu'elles existent. Je ne sais pas dire si c'est lié à des caractères spéciaux ou autres.


I had looked it up and the only issue was that the javax.annotation.Resource annotation in annotation-api contained a new "lookup" field introduced in "Common Annotations 1.1", while the jsr250-api annotation did not have it. I guess there could be a problem if a client gets the older version and expects the new field to be there.

It is probably better to use annotation-api in our code and probably ban jsr250-api with the enforcer plugin, or something like that.


Bitte hier eine eigene Variable erstellen: `@media-text-color`


Er, you can't. Ignore me.


Sounds good. I've got a working reader for Dashing written in Ruby, but it took me some time to figure out what's the metadata I needed. Maybe you could include an option to output the type and code as string?
I would gladly help you with anything I can help with.


バージョンアップ以前にダウンロード済みのものは、イラスト更新日時情報を持っていないので、更新有無の判定はできない。


Yes, I think this is quite trivial, just by patching the lexer.  But there is a risk of breaking code such as   t*#s (which is a valid type expression) if we recognize *# as a token.


Please test and let me know of any bugs before I push this back to master


Should we put all the possible suffixes in one def. only ?


can we pull this match into some common subroutine? It looks precisely the same both here and in the case below?


Yes.  Maybe I should have done that.  It's folding a disjunction (specifically a `TryCorrect[MergePlan]` which is just `Unmergeable \/ MergePlan`) over a list of functions that take a `MergePlan` and produce a `TryCorrect[MergePlan]`.

I assume there is a better way to do this in general though than using the explicit fold?  This must be a repeated pattern.


+1.  I think in this case we shouldn't even worry about the code in the finally block.  This is more of an assertion.  Maybe we should even throw AssertionError instead?


Hm, sure I can. But won't this stop us from reading in any propagator given to us? I mean, part of the idea was that the code should be flexible enough to read in any propagator with only moderate additions to the contraction code. What if we get Nilmani's overlap propagators? We won't be able to change the header easily. I might be missing something, but if this reduces the flexibility of the code, I am against it.

Disregarding cleanliness in favor of flexibility and freedom once more, I think these things should be specified in the input file. Reading from the header can serve as cross-check at most. In the end the user has to be able to control the whole code via the input file.
Let's for argument say, if I wanted to compute in the NREL basis using CHIRAL basis propagators, I should be allowed to do so. In the corresponding routine I would call CHIRAL_TO_NREL and everything should work regardless. This might not seem like an advantage now, but if e.g. I wanted to compute parity projected, non-relativistic mesons or baryons, I would need this.

Hm…I'll continue to think about this, but right now I think the IO should stay as it is and not be further updated.

Cheers
Anthony


Saddest day.


a MATLAB realization about simulated annealing algorithm


Guarda que acá quedo un debugger!


- What demo application is intended for the de2i-150 board? demo_mn_[console;embedded]? I thought demo_mn_console is to be used, which has no buttons/leds... please let me know
- Okay, right use a pipeline bridge with that clock that runs also the Nios / memory controllers in toplevel.


This is putting Nadal as right-handed. Shouldn't this be swapped?


Yo usaria un nombre más descriptivo para la clase: ContactMerger


Disjunction has a `Monad` type class instance so it gets the `>>=` syntax.


Thx!


@bep Awesome!  :clap:


@oberion What do we need these modifications for ? Is this change really necessary to allow compilation on 32bit window machines?? This gives me merge conflicts when merging the new qgc-master.


superfluous semicolon.


Thanks for the update. I think you can simulate the socket disconnect by "service network restart".. but is that too crude?

> I thought the problem was that redsocks kept going down,
Yes, earlier (like in January), I was seeing redsocks goes into "do nothing" mode. The redsocks process itself was running, but when I try to connect to LHCONE site through it,  it just didn't work. Also, I believe one time I was able to fix it simply by restarting the gociptables. In any case, redsocks/iptables seemed very unstable.

More recently though, however, it's the ssh/socks5 tunnel that keeps disconnecting. When this happens, I normally restart both redsocks/ssh, so it's probably that we don't run redsocks long enough to observe the problem I was seeing.

Once we fix the ssh disconnect issue (already fixed via autossh?) I believe we will start seeing the issue with redsocks again, but that's just my guess.


If no song information is available, use `-` instead (Prevents an Python exception). Empty "title"/"albumartist" can occur during stopped state and when no tags are provided.


Print the status code returned by TuneIn for debugging purposes.


Makes sense, thanks for pointing it out. To anyone else reading this issue, this is just a change in the example in the readme and doesn't really affect the library.


Or maybe `AssertionError`?  IIRC I think distinction is that `IllegalStateException` is used more for calling a public method when in an invalid state, where this is more likely an internal programming error.  I'm not sure we've consistently made this separation in the HTTP/2 code, however, so I don't feel strongly one way or another.


je compte si j'ai le temps repasser sur tous les provers ce weel end


Done. I've also refactored the ErrorHandler to lazily instantiate the exception instead of aggressively doing so. Let me know how things look.


@escopecz Was CURLOPT_HTTPHEADER in the if/else statement supposed to be CURLOPT_FOLLOWLOCATION instead as the lines following are setting CURLOPT_HTTPHEADER as well.  Just wanted to confirm.


yo're right. thank you for spotting that.


comments are swapped


[R/simulator_ms.R:32:33:](https://github.com/paulstaab/coalsimr/blob/7dc3ef4e41db27746c701804a2a696eb3d4d5ff6/R/simulator_ms.R#L32) *style:* **Commas should always have a space after.**
```r
               'locus_length', ',')
                                ^
```


Hi,
I have this error when i use your script.

"TypeError: this.drawXAxisLabel is not a function
label = this.drawXAxisLabel(row._x, basePos, row.label);"

Please help me for resolve.

Thank you


add token to user params, comment belongs_to :post


[STATUS] Collision not working well, status set to unstable.


remove this: string() already does the job fine and this method obfuscate the code.


Sorry not following. I'd like to merge this PR. I don't want the docs to refer to things in the mesos build tree that don't exist (lines 89, 93). Since `test-executor` things exist in the mesos tree for Java and Python, that's what we should point to in the docs. I'm fine with the mesos-go versions having `example-` prefixes.


1: M111 unrecognized component name: email


1: M111 unrecognized component name: legacy


Try composer install and composer update if the assets are not loaded by composer.