I think it's ok. All methods, where exception is thrown once doomed resource was detected behaves like it did not exist at all. The very same exception could be thrown by resourceManager.getResourceById(overlord, resourceId);


Looks like the answer is yes.


Thanks for the merge Daniel! I can see this made it into the rendered pythonjobs.github.io repo, but it doesn't seem to have made it on to the live site. Any idea why?


It's returned by `inferBindContext` if it's truthy.


Right. It's time to take care of that. Will change it.


You have bad habbit to copy variables and reuse it :) You don't really need it.


ok


Changing state to props leaves some dead code in the file as far as componentDidMount, and getInitalState.

Also there needs to be a second check of if(this.props.collection.showcases.length > 0) because this.props.collection.showcase can be an empty array (but this check is needed because this.props.collection.showcases.length can be undefined). These changes are all made in #100


Sure.


Why Leo why ?

Return always the same message no matter what you get in params[:id]


Just status 404.


Damn.  It di dpass e2e, though.  Interesting.  Fix coming.


indentation


merge


Why did you change that ?


Closes #3


Test the output.


It can. Let me change it.


Yes, it's easy.


@malufett
I think Playtester meant atk2 filed is  variance not meant  atk2 is  variance
https://github.com/HerculesWS/Hercules/blob/master/src/map/mob.c#L3372


Thanks, I hadn't checked the older tests : I shall cancel mine, and maybe restrict testing to the queen+rook battery.


Closes #2


Improved non-verbose logging of the IPMI server. Added Set User Name Command and unrecognized command handling


Trzeba będzie zmienić php'a żeby tak generował json'a


I just fix the syntax error,maybe there is some other error with the source.Send  me(xyyangkun#163.com(replace # to @ )) the zip, let me try it.


@QQfoolsorellina
Its matk, and atk1 is atk
And there's formula to calculate atk/matk range.


QSTileHolder.java:135: error: ';' expected


In getSupersededBy(), you seem to have switched two lines round for no obvious reason. Am I missing something? Also, can you use an empty map, or can it not be immutable?


Basically the philsophy is to provide a foundation for implementing more specific stuff.. at some point there will be more abstract drivers on top of the GL driver


what is difference between omniarmv6 and androidarmv6 ?


Changed URL to new Tango Card Enterprise Terms of Service Agreement


s/DENING/HARDENING/ ?


msg.match[1] returns the "youtube" parameter instead of the actual query.
Should be msg.match[3]


Just use gst_element_factory_get_metadata() here with the key GST_ELEMENT_METADATA_KLASS


It was visible to me in non-GM mode (with GM account). But my versions of Core and DB are not identical to github. Could you please explain why this rabbit must be invisible without invisibility aura? What is the difference between it and other creatures?


And not using Global vars would be best! =D Global vars are evil!! =)


We usually do strstr(classification, "Visualization") here :)


No need to explictely load it here


NOTE

Instead of those 3 parameters we could require a CKTransactionalComponentDataSourceConfiguration, however this sounds a bit leaky right now. Should we make CKTransactionalComponentDataSourceConfiguration be only the configuration related to component generation (which it is right now) and rename it accordingly so that it is not specific to the CDS (e.g: CKComponentGenerationConfiguration, CKTransactionalDataSourceComponentGenerationConfiguration) ?


Hey @Thomoore, there's actually a [`tel` field type](http://getkirby.com/docs/cheatsheet/panel-fields/tel)!


I cheated and added some `?:` flags in the regex. It makes it ignore the pattern match.


Ah ok.


TODO

Full mount process


Well, since you're using Mendeley, you should first try Mendeley support: http://support.mendeley.com/ . Otherwise, if that doesn't help, we can indeed help you on the Zotero forums (via the link @zuphilip gave you). But whatever support channel you pick, you should provide more details. "some errors" is not specific enough. What is the exact error message you get?


Then everything could be a bit simplified:
```javascript
function getStaticContext(bind, scope) {
  var object = bind.object || bind.callee.object;
  return scope.isStatic(object) && object;
}
```


gst_registry_get_feature_list(gst_registry_get(), GST_TYPE_ELEMENT_FACTORY)


TODO

- Bounds animations
- Better state bookkeeping, notably initial state


See my comments :) And this seems a bit over-engineered. Just have an API in GstPlayer to get the list of all available vis element factories names and descriptions, and then allow to set them by name on GstPlayer. No need for a new file or anything :)


1. no need to make it non-nullary method - parens are not required, because the method is not intended to cause a side-effect (I wouldn't count memoization of all hosts as a side effect, even though, technically this method is impure). Non-side-effecting 0-arg methods don't have parens in scala.
2. I think if it is a good idea to make it a `lazy val`. This way it would be guaranteed to always return the same set of hosts.


I made this commit


I'm not fully aware of what is in 15.05 - but you are currently targeting ``dev`` of Galaxy not ``release_15.05`` so I feel like it is fine to target ``dev`` for now - once Nate branches ``15.07`` (mid-June probably) - then would should pin things down more specifically in that branch?


Are clients supposed to be expecting both formats right now?


So here you could just return a const GList* of all the visualization element factory names for example. Or some new struct that contains name and description.

But it should be (transfer none) and you cache the list internally. See the plugin feature list caching inside gstplaybin2.c or gstdecodebin2.c for example


Really ? I just tested it and it was always returning the same video until i changed the index to 3.


Keep the second parameter NULL


You don't have to change anything here, but some style nits for future reference:

* git commit subjects are supposed to be limited to 50 characters
This is useful for browsing commits with git utilities that only show 50 characters of the commit message (and you can see here that github displays it kind of ugly too.)
* git commit subjects are supposed to be written in the imperative
s/Adding/Add
This consistency is a reading aid for users browsing commits in the future


Should be const


No good point silly mistake. This was an oversight and corrected in next push


@Snetzky no problem! Take your time! :)


Not the name, but the element factory name is what is needed here


It's unnecessary... because of rspec failed reason.


It is a shame this was never merged in / published, but I am thankful I was at least able to find this commit which had a better explanation than the published docs.


Why did we add this to the base spec? Seems like it should be opt-in


Odd. Wonder if that's a later feature in Coffeescript. My local ones worked. Another reason we should get this under CI. :smile:


You need to add .md extension to the file, otherwise it will not be rendered


:+1: for CI !


Won't this still flip-y if spec has {:flip-y false} ?


I have to say this is still pretty odd. I would expect `defaultTriggers` to be a `List<EmailTrigger>`, rather than relying on `createDefault()`.


@dbnicholson Your explanation here is exactly what I was thinking. I'll update the commit message to clarify this.


bootstrap has dependencis. I hope this info will help: http://sourceforge.net/p/moneymanagerex/wiki/Building%20mmex%20Debian%20%28Ubuntu%29%20Linux/

Error:
./bootstrap.sh: line 4: bakefile_gen: command not found
./bootstrap.sh: line 7: bakefilize: command not found
aclocal: error: configure.ac:53: file 'm4/autoconf_inc.m4' does not exist
configure.ac:45: error: possibly undefined macro: AM_OPTIONS_WXCONFIG
      If this token and others are legitimate, please use m4_pattern_allow.
      See the Autoconf documentation.
configure.ac:46: error: possibly undefined macro: AM_OPTIONS_WXPRESETS
configure.ac:48: error: possibly undefined macro: AM_WXPRESETS_FULL
configure.ac:53: error: possibly undefined macro: AC_BAKEFILE
configure.ac:53: error: possibly undefined macro: m4_include
configure.ac:61: error: possibly undefined macro: AM_WXPRESETS_MSG_BEGIN
configure.ac:62: error: possibly undefined macro: AM_WXPRESETS_MSG
configure.ac:63: error: possibly undefined macro: AM_WXPRESETS_MSG_END
The command "./bootstrap.sh" failed and exited with 1 during .


references #1


@ProgstarGrixisLhz Why upload a _GitHub file?What is the different between this two file?


develop (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1140 of 1140 tests passed)

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


@cristipp @schmmd @jkinkead ?


Clients should change to use this new key as it makes sense for them to do so.

Ideally I think you would try to resolve all server-unresolved content from within the app (local intranet), or attempt to fetch metadata another way (mdns), leave results out entirely, or just dump them all into a "who knows" section.


Thanks for the catch,  legacy when I was just looking at video.


In theory it could not happen but better safe then sorry. The check was put in because consistently would get notice to switch the the same index.  So a check that < will work fine and has been added.


It doens't work. Need to work another way


Very minor, specially since there aren't any nested loops here, but consider using more meaningful index name.


@trylimits writing page table entries are fixed. now all page tables are written correctly. nonetheless, pabts will continue to occur. I think the reason for this is that the link register in the dabt handler needs this lr_dabt - 8 I showed you today (see p. 122, "5.9 data abort handler" at ARM Developer Suite v1.2 ). I would be great if you an @Blackjack92 would have the time to try this out tomorrow. I've prepared everything you need so far.


Some times they untie themselves


its a bug


you query the table's name in a few places, is it worthwhile to assign to a local var?


Hey, I'm waiting on Hannah Greenberg to send me her information. She wasn't included on the site because she wasn't on the Google Doc file. I also need to capitalize "briana marquit"s name. Is there someway you can grant me access to edit these files without your approval?

Thanks, Shan


The user has to wait until the e-mail is sent *to* get the response of the registration request. *When* using queues you would just *delay* the e-mail delivery to background


I see. You're saying that https://github.com/cubing/tnoodle/pull/174 has everything we need? If that is the case, could you look through my comments on your pull request? I had some minor variable naming/readability things I'd like to resolve before we pull in your changes. Thanks!


Minor - fix indentation


Nice! I was wondering why `fs` was always defined when I opened the console.


king -> kind


##Milestone 3 Feedback
Great job Ella. Your focus is concise, and your wireframes clearly communicate your direction going forward.
Nice use of online wireframing tools. It looks like the code you have on GitHub for this milestone is behind the code that's up on ellahaines.net. Have you pushed all your latest changes? Overall - great work. Your Milestone passes at Tier IV.

###Suggestions
Begin with clean, solid markup, and use the built-in Bootstrap functionality to the greatest degree possible to streamline your development process. After this, then add the bells and whistles. Keep your large background images under 500kb, and you'll need to use @media queries in your CSS to load smaller background images for smaller screen breakpoints. Keep these small in both dimensions and file size.

The layout structure you have here in your wireframes should be easy and straightforward using Bootstrap, for both your mobile and desktop views.

Look at the swipebox plugin for your gallery here: http://brutaldesign.github.io/swipebox/
A previous student had great results with it.


subscription payment -> subscription plan?


rax retry tests


Yeah, this caused some missing requires to not be detected in the metrics specs


@agarie you can see the old API tests here.


Derp, reverted this :(


And here.


Are you sure it is the right date?


incluidos los archivos "jquery" & "jquery_ujs" ya que ocasionaba conflicto con rails para el método delete


where is test 1 ?


Common. Say something about yourself


There was a reason I didn't do it that way, but I don't remember what it was. I have to have a createDefault at some level though because the triggers themselves can have a different constructor for additional UI elements. So, I can't just call the constructor with known parameters, especially since other plugins can provide EmailTrigger implementations via Extensions.


Not lame but sad...


using their social networks -> with their social network accounts?


noted


https://twitter.com/lambdasawa/status/598495602444382208 というのを見かけたので指摘を。

- applyが末尾でない再帰なので、`as` の長さが長いとstack overflowする
- `as.tail` がそれほど効率よくないもの(たとえばVector)の場合は、無駄な中間オブジェクトがある程度発生する？


This is the safe zone... you can tell something about yourself


Don't patronize yourself Lol. Looks like it breaks all 5 SOLID principals. At least it's consistent.


Yes, so this chunk of code executes during the connection phase before any
testcase actually starts. So immediately after this exception, teston will
bail out.

On Wed, May 13, 2015 at 4:39 PM, Jon Hall <notifications@github.com> wrote:

> TestOn may actually handle exiting the test later on. maybe you just need
> to set self.componentDictionary[component]['host'] to None and TestON will
> fail the test later in this method
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/opennetworkinglab/ONLabTest/commit/fa93ecf30ff99f1cd1a46120cd64044786c24e3c#commitcomment-11187403>
> .
>


Random Colors not Gradient.


@anupreeta27 few minor edits according to previous discussions
(i) Known samples: sentence added
(ii) harder trajectories: slight rewording to make it easier to read
(iii) minot rewording


If you're looking for an exact match, why not just do a hash look-up rather than iterating through the table?


@Emile-Filteau So, just out of curiosity, what version are you running of coffeescript? I cannot find a change log entry where the "[non-capturing group](http://www.regular-expressions.info/brackets.html)" flag was added into (or possibly broken) in their regex parser.

```
$ coffee --version
CoffeeScript version 1.9.1
```


Submit pull request now... please delete the branch when merged... thanks!


Does case matter?


Hello,

I've installed PDF Report in SonarQube in version 4.5.4 and PDF Report in version 1.4. The pdf generation is initiated at the post job task by sonar-runner.

To have a report, I made sure to:
- Setup is configured to generate the report; and
- Project dashboard has the PDF Report widget visible.

I've run into some trouble for having force authentication with LDAP turned on, but if you remove the force authentication option everything seamed works fine.

PDF Report in SonarQube version 4.5.4 is not generating the report due to an error while executing Sonar runner. During post job execution the error raised says the user is not authorized by throwing a 401 HTTP STATUS CODE.

```
ERROR: Error during Sonar runner execution
org.sonar.runner.impl.RunnerException: Unable to execute Sonar
at org.sonar.runner.impl.BatchLauncher$1.delegateExecution(BatchLauncher.java:91)
at org.sonar.runner.impl.BatchLauncher$1.run(BatchLauncher.java:75)
at java.security.AccessController.doPrivileged(Native Method)
at org.sonar.runner.impl.BatchLauncher.doExecute(BatchLauncher.java:69)
at org.sonar.runner.impl.BatchLauncher.execute(BatchLauncher.java:50)
at org.sonar.runner.api.EmbeddedRunner.doExecute(EmbeddedRunner.java:102)
at org.sonar.runner.api.Runner.execute(Runner.java:100)
at org.sonar.runner.Main.executeTask(Main.java:70)
at org.sonar.runner.Main.execute(Main.java:59)
at org.sonar.runner.Main.main(Main.java:53)
Caused by: org.sonar.wsclient.base.HttpException: Error 401 on http://127.0.0.1:9001/api/issues/search?pageSize=20&componentRoots=MYCODE&rules=findbugs:ODR_OPEN_DATABASE_RESOURCE
at org.sonar.wsclient.internal.HttpRequestFactory.execute(HttpRequestFactory.java:153)
at org.sonar.wsclient.internal.HttpRequestFactory.get(HttpRequestFactory.java:129)
at org.sonar.wsclient.issue.internal.DefaultIssueClient.find(DefaultIssueClient.java:49)
at org.sonar.report.pdf.builder.RuleBuilder.loadViolatedResources(RuleBuilder.java:105)
at org.sonar.report.pdf.builder.ProjectBuilder.initMostViolatedRulesFromNode(ProjectBuilder.java:248)
at org.sonar.report.pdf.builder.ProjectBuilder.initMostViolatedRules(ProjectBuilder.java:175)
at org.sonar.report.pdf.builder.ProjectBuilder.initializeProject(ProjectBuilder.java:98)
at org.sonar.report.pdf.PDFReporter.getProject(PDFReporter.java:132)
at org.sonar.report.pdf.PDFReporter.getReport(PDFReporter.java:82)
at org.sonar.report.pdf.batch.PDFGenerator.execute(PDFGenerator.java:109)
at org.sonar.report.pdf.batch.PDFPostJob.executeOn(PDFPostJob.java:70)
at org.sonar.batch.phases.PostJobsExecutor.execute(PostJobsExecutor.java:72)
at org.sonar.batch.phases.PostJobsExecutor.execute(PostJobsExecutor.java:61)
at org.sonar.batch.phases.PhaseExecutor.execute(PhaseExecutor.java:132)
at org.sonar.batch.scan.ModuleScanContainer.doAfterStart(ModuleScanContainer.java:194)
at org.sonar.api.platform.ComponentContainer.startComponents(ComponentContainer.java:93)
at org.sonar.api.platform.ComponentContainer.execute(ComponentContainer.java:78)
at org.sonar.batch.scan.ProjectScanContainer.scan(ProjectScanContainer.java:233)
at org.sonar.batch.scan.ProjectScanContainer.scanRecursively(ProjectScanContainer.java:228)
at org.sonar.batch.scan.ProjectScanContainer.doAfterStart(ProjectScanContainer.java:221)
at org.sonar.api.platform.ComponentContainer.startComponents(ComponentContainer.java:93)
at org.sonar.api.platform.ComponentContainer.execute(ComponentContainer.java:78)
at org.sonar.batch.scan.ScanTask.scan(ScanTask.java:64)
at org.sonar.batch.scan.ScanTask.execute(ScanTask.java:51)
at org.sonar.batch.bootstrap.TaskContainer.doAfterStart(TaskContainer.java:125)
at org.sonar.api.platform.ComponentContainer.startComponents(ComponentContainer.java:93)
at org.sonar.api.platform.ComponentContainer.execute(ComponentContainer.java:78)
at org.sonar.batch.bootstrap.BootstrapContainer.executeTask(BootstrapContainer.java:173)
at org.sonar.batch.bootstrapper.Batch.executeTask(Batch.java:95)
at org.sonar.batch.bootstrapper.Batch.execute(Batch.java:67)
at org.sonar.runner.batch.IsolatedLauncher.execute(IsolatedLauncher.java:48)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:601)
at org.sonar.runner.impl.BatchLauncher$1.delegateExecution(BatchLauncher.java:87)
... 9 more
```
The pdf is generated as a post job task by sonar-runner that reports the user is unauthorized.

I use LDAP to authenticate my users and force authentication on the server. I've entered a valid Administrator user and password for the keys sonar.pdf.password and sonar.pdf.username and tested them manually by both using them in CURL and in a browser. If I remove the force authentication option everything works fine.

What am I missing here?


should probs consider using phusion baseimage

site basically explains why
https://phusion.github.io/baseimage-docker/


Ok, this is still broken.  qz-websocket assumes the insecure websocket is running on 8182 so it never finds it if the secure websocket startup fails.  I'm going to put some time into this.


is it possible _tables was not empty when passed in?


Haha just got introduced to those today

> On May 14, 2015, at 12:04 PM, Ransom Carrolll <notifications@github.com> wrote:
>
> Don't patronize yourself Lol. Looks like it breaks all 5 SOLID principals. At least it's consistent.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/mikepitre/Blackjack/commit/993df8365278c7fe192d1fbd7e18bb15933b7549#commitcomment-11199081>.
>


More info?


not sure, open a bug? Since this commit is already on master.


You're right, it was not needed. I wanted to use ResourceFlyweight and once I find out it does not fit my needs, I just made more lightweight copy of it.

parentResourceId stuff is not used, but I wanted to use it when I'll be fixing https://github.com/rhq-project/rhq/blob/master/modules/enterprise/server/jar/src/main/java/org/rhq/enterprise/server/resource/metadata/ResourceMetadataManagerBean.java#L633 (there's no BZ yet, but it could be very slow)

I agree, I'll rewrite this using named queries (same with parentResourceId stuff later)


これもstack overflowする。末尾再帰だが、finalがついてないので、最適化されない


Done.


i uploaded on w web site but now it error on luine 26 where the address been written and i cant find a proper address of .py file

Date: Thu, 14 May 2015 08:23:55 -0700
From: notifications@github.com
To: cookie_stealer@noreply.github.com
CC: faizanharoon@hotmail.com
Subject: Re: [cookie_stealer] Cookie logger PHP script (b8dc28b)

You don't have to upload it on github. It should be uploaded on w web host.

Read the description properly.

On 14 May 2015 20:52, "Pakheria" <notifications@github.com> wrote:

> how do i upload it at github

>

> Date: Wed, 13 May 2015 10:23:25 -0700

> From: notifications@github.com

> To: cookie_stealer@noreply.github.com

> CC: faizanharoon@hotmail.com

> Subject: Re: [cookie_stealer] Cookie logger PHP script (b8dc28b)

>

> or just copy the code from this url. It will work fine.

>

>

>

> https://github.com/rash2kool/cookie_stealer/blob/master/cookie_logger.php

>

>

>

> On Wed, May 13, 2015 at 10:45 PM, Rashid Feroz <rashid.2008feroz@gmail.com

> >

>

> wrote:

>

>

>

> > ok. .just remove the '+' sign from the beginning of each line. It would

>

> > run perfectly then. Github might have added that.

>

> >

>

> > On Wed, May 13, 2015 at 8:36 PM, Pakheria <notifications@github.com>

>

> > wrote:

>

> >

>

> >>

>

> >> no i didin't remove any thing

>

> >> Date: Wed, 13 May 2015 02:00:08 -0700

>

> >> From: notifications@github.com

>

> >> To: cookie_stealer@noreply.github.com

>

> >> CC: faizanharoon@hotmail.com

>

> >> Subject: Re: [cookie_stealer] Cookie logger PHP script (b8dc28b)

>

> >>

>

> >> Just remove that if statement with the brackets.. It will work even

> then.

>

> >> well, did you remove the '+' sign that is at the beginning, from every

>

> >> statement?

>

> >>

>

> >>

>

> >>

>

> >> —

>

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

>

> >>

>

> >>

>

> >>

>

> >>

>

> >>

>

> >>

>

> >> —

>

> >> Reply to this email directly or view it on GitHub

>

> >> <

> https://github.com/rash2kool/cookie_stealer/commit/b8dc28babbf9d9a8fa9d7cd8124eff4af8cf2cdd#commitcomment-11177940

> >

>

> >> .

>

> >>

>

> >

>

> >

>

>

>

> —

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

>

>

>

>

>

>

> —

> Reply to this email directly or view it on GitHub

> <https://github.com/rash2kool/cookie_stealer/commit/b8dc28babbf9d9a8fa9d7cd8124eff4af8cf2cdd#commitcomment-11197072>

> .

>

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


Predict counts from 00:00:00 UTC (see e.g. Fraction_of_Day on line 560 in predict.c or CurrentDaynum() on line 3263).


Lol, didn't see NFOs for such a long, long time... Thanks for rescuing me from nostalgy!


Immediate mode only starts when the start bit transitioned from 0 to 1, not setting inmediate mode itself. DMAs get disabled effectively by switching a running DMA to immediate mode.


variable should be underscored: `syntax_file_js`


Different clients use different symbols for wildcard (I think % is commonly used). Also, all wildcard symbols should be translated to * for WsSQL.


Can you please extract `App::make('translator')->getLocale()` to a protected `locale()`?

Thanks


Too much Bio


There are several magic constants in refraction.c. Can we name them? Or is this code take from predict so that we don't know?


TeamCity xBIM :: Xbim Geometry :: Release [Build 59](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1628&buildTypeId=XBIM_XbimGeometry_Release) is now running


Agora sim!!! :D


Fixed in commit c0ae177d9e96b190801ac949daef057d2db79d87.


Do you know that for sure? Do you have a unit test to prove it?

See `SoapCommands.GetCrudItems()`


Faltou o import do "List".


What is the reason for checking for a particular exception as opposed to
just using "except: "? Also, why would I want to catch a ctrl-c keystroke,
and what would that syntax look like? Sorry, I am not all that well
aquatinted with exception handling.

On Thu, May 14, 2015 at 10:04 AM, Cameron Franke <cameron@onlab.us> wrote:

> Yes, so this chunk of code executes during the connection phase before any
> testcase actually starts. So immediately after this exception, teston will
> bail out.
>
> On Wed, May 13, 2015 at 4:39 PM, Jon Hall <notifications@github.com>
> wrote:
>
>> TestOn may actually handle exiting the test later on. maybe you just need
>> to set self.componentDictionary[component]['host'] to None and TestON will
>> fail the test later in this method
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/opennetworkinglab/ONLabTest/commit/fa93ecf30ff99f1cd1a46120cd64044786c24e3c#commitcomment-11187403>
>> .
>>
>
>


please use active test here


I got it figured out.... Sending you an e-mail


Should this be `retrofit-adapters`?


Messed his name up. Whoops!


I use version 1.9.2.
When tested locally with msg.match[1] i always get this video : https://www.youtube.com/watch?v=GexOspF-Kr4
I tested it with a some different adapters too and I have the same result.


Egalement mise à jour des champs clé dans la DB


This is important: node numbers initialised to -1 by default for not-null checking...


I stand corrected. I'll get this pulled in.


:crying_cat_face: sigh


The team consensus appears that it's OK to have it the default, just like
JUnit does. We settled for providing an alternate Spec if we really really
don't want it for some reason, but there was no need for such Spec so far.

https://groups.google.com/a/allenai.org/forum/#!search/Scala$20Testing$20Adventures/dev/1g-f9iICFpA/T8jcJp2FXk8J

Looking forward reopening this bikeshed ;)

On Thu, May 14, 2015 at 9:54 AM, Vu Ha <notifications@github.com> wrote:

> @cristipp <https://github.com/cristipp> @schmmd
> <https://github.com/schmmd> @jkinkead <https://github.com/jkinkead> ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/allenai/common/commit/11c355a756c18bd0bff90c30c9179833dc09dbd9#commitcomment-11198897>
> .
>


An update to `<solarized>` has broken this emacs setup by removing support for the separate `<solarized-light>` and `<solarized-dark>` themes.


That's the spirit! :birthday:


Refactored TreeBranch[] array to ArrayList since we don't have a good way to tell how many nodes beneath it so we'd have to dynamically resize anyway at every node traversal... easy enough to convert the array list returned by the top node to the calling thread using toArray() anyway.


Fixed in bdf7fc683b8b49b11bb40bcfac8ffb9853bf9799,


would a filter such as thor::myfiles::tests::* cause the driver to fetch all files?


No idea how this survived this long


If something cannot happen you should put a `panic`. `panic` is the golang way of asserting. Same in `detectEndpointTarget`


:sparkles:


Thanks. Again I really appreciate this library and all your work on it.


Hi, It's only a minor comentary: 'extrapoltion' by 'extrapolation'. Cheers.


Went with compile in stead of aggregate for clarity.


The name implies only cached tables are retrieved, but should we be retrieving all tables available?


Review complete.


> but there was no need for such Spec so far

what about this? https://github.com/allenai/scholar/blob/master/offline/pipeline/src/it/scala/org/allenai/scholar/pipeline/spark/BuildIndexSpec.scala#L9

Looks like a case where you don't want this.. but there is no way to still extend our base spec without it


Yep! 15.05 is without ENV, so https://github.com/galaxyproject/galaxy/blob/release_15.05/config/plugins/interactive_environments/ipython/config/ipython.ini.sample is correct. For 15.07, exactly, we'll push a new image.


Perhaps it's an oddly specific search term? It appears to be using it.

```
user> !yt star wars
hubot> https://www.youtube.com/watch?v=ngElkyQ6Rhs
user> !yt star wars
hubot> https://www.youtube.com/watch?v=222-yrMLZFs
user> !yt star wars
hubot> https://www.youtube.com/watch?v=mDeJjBDlWBc
user> !yt star wars
hubot> https://www.youtube.com/watch?v=mDeJjBDlWBc
user> !yt star wars
hubot> https://www.youtube.com/watch?v=LhJA7I_8-14
```


You should error out for the default case


@bgruening do we need to make a PR to release_15.05 to target our new 15.05 tag rather than 0.2?


I like it. Changed the comment.


##Milestone 3 Feedback
Awesome job - I'm looking forward to seeing The Code Wizard come together! You Milestone 3 passes at Tier IV. For the sake of MVP - let's go forward with layout one for now. I do look forward to seeing the full desktop site some time in the near future!

###Suggestions
Begin with clean, solid markup and structure, and use the built-in Bootstrap functionality to the greatest degree possible to streamline your development process. After this, then add the bells, whistles, and 15 pieces of flair. You've got some awesome ideas here, and a lot of work ahead - so be prepared to streamline if necessary.

One concern I have is your footer layout on mobile. I read recently on UXPin that the ideal touch screen tap target size is 72px - so perhaps the links should be laid out to stack, rather than being horizontal as they are in the wireframe. Just something to think about. Bootstrap has built-in popovers that might work for the footer: http://getbootstrap.com/javascript/#popovers. But if these don't end up working out (they probably won't with the contact form), perhaps look at the modal windows as an alternative. Example here: https://bootcamp-coders.cnm.edu/classmaterials/week6/bootstrap-sample-modal/

Also, look at the jQuery slideToggle() for those sections of code you want to feature. Here's a quick example here: https://bootcamp-coders.cnm.edu/~rlewis37/bootstrap-sandbox/


I thought of that, but I was bit hesitant in adding a panic().
At the end are still handling that condition gracefully.

If you are ok with it, I can change it to panic().


Yes, but we voted to get rid of rule 2. There is no reason to not allow blank lines inside of functions.


Tested this in the code and a also in some sample code and File(String, String) handles relative paths.


I'll admit that I fail to see which test that fixes.


I think before releasing 1.1 to the chrome store, you may need to update the migration process to include 1.1, otherwise users that jump past 1.0 will not get the full monty.


@tomekwi Actually, the lack of a $ on line 35 is the typo.  If you don't provide a function to use for injection, sinject will look for a function named $inject on the object.  Why the $?  As an indication that the function is "special", and because $ => s, $inject => sinject.

Good catch.


Thanks, will take care of it.


True.we can folk this there and delete them from here


no. this was a test. forgot that github is adding reference to bug is listed in comment. ignore this please


I think this the root cause. I posted it here

https://github.com/Netflix/exhibitor/issues/254


In GM mode summoned pirates could never attack GM-player, but they did it and only after that I noticed the fluffy rabbit (which is much better than Summoned Infernal model used for triggers in many cases).


@tschoonj Can you submit a PR to remove the need of `pkg-config` in the test? Thanks.


Customer bliver aldrig created med alt info på en gang. Resten af info om en customer bør ske med setters.


You can make this a list of `[]*networkResource` as well


Agreed.
Improved in https://github.com/blueyed/neovim/commit/a94e658d49d2679b2a8c72941ba4a74905b96f18

Please note however, that errors are not displayed currently anyway (https://github.com/neovim/neovim/pull/2549#issuecomment-98418657).


Catching `Exception` is almost never a good idea. We should only catch specific exceptions like IOException, WebException etc. that are known to happen. Things like NullReferenceException which happened here - should really never be caught as they often indicate bugs in the code. The semantics for plugin developers should be that all unhandled exceptions should result in a known `PluginResult` returned by the framework hosting the plugin i.e. cordova bridge and not require every plugin method to "handle" all exceptions.


No problem


Btw +1 on removing async here as this operation already happens on a threadpool thread and there is no value in making it async. In fact, making it async can result in issues with more than one of these happening at the same time.


Cool


It can work either on the CPU through numpy or on the GPU by converting to float32 before calling exp() like all other elemwise operations.

I'll change the threshold.


I suppose the issue is mostly from Zotero's side. E.g. https://github.com/zotero/zotero/blob/4.0/chrome/content/zotero/xpcom/citeproc.js#L13512 citeproc-js (reasonably) expects that the ID it passes to `retrieveItem` is going to be the ID it gets back in `Item.id`, so the ID it uses in `state.registry.registry` is the Zotero database ID. Later it uses `Item.id` to access the item in `state.registry.registry`, so this doesn't work if we change `Item.id` to be the item URI.

I suspect that if Zotero item picker were to pass back item URIs, instead of DB IDs to citeproc to begin with, this issue would go away, though I haven't tested it yet (leaving this change for the next PR).

My only concern at this point is regarding existing embedded references in the documents. In general, I'm not familiar with the citeproc-Zotero interface, so I'm not really sure what breakage to expect. I'll look more into it at some point.


谁告诉我 Linux 下怎么找IP。。。  真心太慢了 一直要刷新刷新刷新


loses order. not sure that bothers me, but if the API says it's a list, and I create `["foo","bar"]`, and get back `["bar","foo"]`, is that disconcerting?


My only final nitpick in the whole patch (sorry!) -- s/argements/arguments/


oh, nevermind, this is only used for non-persisted users


What's better is that the default template is jade - w00t


Basically this sets us up for a basic API


I should note that this only became apparent at very low throughputs, when the next fetch request is fired before the next message has been produced. If the next message had already been produced by the time we attempted to fetch it, we'd be missing one message per fetch and masking this error.


server config file


It is perfectly fine to leave this like this:

```python
if preferences[style]
```
It already defaults to `if True` in any case.


Looks good


Maybe sth like PositionIsAvailableForSpawningMonster ?


Wow, that sucks. Thanks for the patch. I'd like to get @hynekcer's input, as I didn't even remember SOQL supporting sub-queries like that in the first place.

What was the ORM line you were using that generated this error?


ha!  no worries!  thanks


Good point.
Changed in https://github.com/blueyed/neovim/commit/c824023d64115cb09b1e5eb7f74358a127782278.


info not warn?


what happens if you do the following in Python

```python
if not []:
```

or

```python
if not drink:
```

In your terminal (python console) you can check the following

```console
>>> not []
```

`not []` defaults to `True`; it is also a test for emptiness.


import base


Great job mate!!! :+1:


Makes sense. Will take care of it.


@cfallin FYI, if you make a comment on a commit, vs a PR, when I rebase and force push, the comments will be lost on github.  Not the end of the world, esp. in this case I'm sure :), but FYI


Sorry for the delay
Please add comments in the code about the issue you have fixed


Awesome—so glad prose is working for you!


Do you plan on updating the streaming API to use HBase 1.0 Client API?


TIleIsFree ?


Caused weird bug in Firefox and affected the Tumblr UI elements not in iframes (namely, the lightbox), so removed.


I was initially against this, but then I got schooled by Christian, so I don't get opinions anymore. ;)

I think this is a reasonable default, but we should throw up a `SingleInstanceUnitSpec` somewhere.


lol. Love it.


Fixed in https://github.com/neovim/neovim/commit/a7f23958b820e9e51e9b53fbabd60c35e135cc15 (with some additional clarification).


Might wanna kill this!


Obsolete, as all images in the `article`s have `width` set on them. Was also causing bugs with Tumblr's lightbox.


Are any of these tech terms we should link?


@sudosurootdev Please, see this log. I have systemUI force close on startup since I have updated the sources.

http://hastebin.com/oduxidalor.coffee


+1 on the code but we should really add
>If you want to register a custom converter creating
>UDT values, you must register a converter converting to connector's
>UDTValue class, not Java Driver's UDTValue. This way it would be
>properly selected and used by the RowWriter.

 to the docs


It's killed


Thanks @avalenzuela00


Good catch. I didn't see that one. :)


Ah, alright. Thank you for the explanation :)


Not sure I was clear, but I also vote +1 for making method call sync. Regarding try/catch I still think that more local try/catches and more meaningful error messages are better, but I'm totally fine merging this w/o adding try/catch - this is just my recommendation to make code more reliable (current crash is an example of this).


Need to change again documentation to state that the input param is a conversion function, not a type imo.


Nikhil - the guidance that the .NET team issued about not catching Exception was redacted between .NET 2.0 and 3.5 timeframes.  Initially, they said "all application-sourced exceptions should derive from ApplicationException", with the theory that you could catch ApplicationException.  That ended up being much more annoying than it was worth, and so the rule about not catching just "Exception" ended up getting relaxed when they introduced critical execution paths (and reliability contracts; see Constrained Execution Regions).

I also don't really understand your point about it being async.  If the 'CopyCookiesFromWebBrowser' method is async, using the C# async semantics don't make it more async.  It's just syntactic sugar.  -1 on removing async.  The 'CopyCookiesFromWebBrowser' method, if it returns a Task, is going to run on yet another thread anyway.  The language support doesn't spin up a new thread; it just means "you can use await semantics rather than doing a Thread.Join", which is what you ended up doing, which is less readable.


Nice catch :+1:  Cool :cherries:


OK, reading through the citeproc manual, I see that all of that is on Zotero's side. Sorry for drawing your attention to this unnecessarily.


Doesn't `parent_location=self.vertical2.location,` above make this irrelevant?


this is already added on line 1


"thing"... Then call it object :D


This should be `project_task_type_done.id`


Don't browse a recordset! self.env.ref already returns the recordset.


You don't need this instruction now


ack


I chose the same norm because we perform computations in float32 anyway and because `numpy.float16(1.0/(2**31+65)) == 0`.

So we get random 32 bit floats that we then downcast to 16 bits.  This will probably oversample 0.0 a lot.  I'll make an attempt at fixing it.

In 64 bits we get a similar but fairer situation in that we essentially generate a 32 bit float and then upcast it to 64 bits.  This has less bias, but will never generate very small numbers.


@KarishmaBagga The change as such looks good. But the identation seems messed up. Make sure you use spaces for tabs in your editor. Also, make sure to run grunt from the dash.js/build directory to ensure it passes.


@andrewducker The code is missing some backticks after `-Key, -BriefDescription, -LongDescription` so it fails when I try and load it manually.


@KarishmaBagga Not sure why your rebasing did not push your changes to the top over the dash industry forum changes. Make sure when you rebase our changes are replayed over all of the dash.js changes.


assuming your test passes, ack


I think if you implement these functions you can test them too. No need to wait until Bear implementation.


Hello @philchristensen, it was on this snippet:

```
class OpportunityLineItemQuerySet(query.SalesforceQuerySet):

    def has_schedule(self, opportunities, has_schedule=True):
        return self.filter(
            has_schedule=has_schedule,
            opportunity__in=opportunities
        )
```

The filter that caused everything was indeed `opportunity__in=opportunities`


haah


--ipconfig

jglimmers <notifications@github.com>于2015年5月14日星期四写道：

> 谁告诉我 Linux 下怎么找IP。。。 真心太慢了 一直要刷新刷新刷新
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/goagent/goagent/commit/98eca96b72bad5bc130c52aa8ef360651b429221#commitcomment-11199617>
> .
>

--
Send by Gmail for iPhone
通過iPhone版Gmail發送
per Gmail für iPhone
ارسال شده توسط Gmail برای iPhone
envoyer par Gmail pour l'iPhone
отправить по Gmail для iPhone
아이폰 용 Gmail로 보내
uenire per gmail for iPhone
iphone用のGmailでお送り
enviar por gmail para el iPhone
לשלוח ידי Gmail עבור iPhone
inviare via gmail per iPhone


We don't need function get_path() and SUGAR_LINK, right?


If you switch to task.Wait() you need to add:

if (task.IsCancelled || task.IsFaulted) and propagate the error / non-error state, and get task.Result.  You're losing the exception as written.  await helps propagate that up.


STEPS, MONITOR_RATE, SAVE_DELAY should be _private


Yes, there is. Without those one space characters the list was shown as a normal text instead of a code block.


@jundl77 this cause the activators to go into an never ending loop....is this the correct fix? I honestly don't understand what you coded. Got confused with the types?


BuildIndexSpec is a great example of DYI when we really really need it. I
was too lazy to link it myself, thanks for bringing it up. I like Jesse's
idea of "SingleInstanceUnitSpec". Or "SharedInstanceUnitSpec".

On Thu, May 14, 2015 at 10:41 AM, Jesse Kinkead <notifications@github.com>
wrote:

> I was initially against this, but then I got schooled by Christian, so I
> don't get opinions anymore. ;)
>
> I think this is a reasonable default, but we should throw up a
> SingleInstanceUnitSpec somewhere.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/allenai/common/commit/11c355a756c18bd0bff90c30c9179833dc09dbd9#commitcomment-11199754>
> .
>


Updated a custom adapter to display sensor names in list


> SingleInstanceUnitSpec

+1


I think you can use Build.VERSION_CODES.ICE_CREAM_SANDWICH instead.


Can you please comment on the pull request as opposed to the
specific commit? You can do so by clicking on the "Files changed" tab
when you open the pull request.

Otherwise, these comments stick around
forever attached to the commit as opposed to the pull request, which can
cause confusion.

On 2015-05-13 09:24, Anna Lorimer wrote:

> Copy and
paste the abstract here?
>
> --
> Reply to this email directly or view
it on GitHub [1].

Links:
------
[1]
https://github.com/wics-uw/website/commit/dc18b0bcc41f73510b060500c03a642afc930de6#commitcomment-11175607


##Milestone 3 Feedback
Excellent work, James. Your wireframes are detailed and clear, and effectively communicate your vision going forward. Your Milestone 3 passes at Tier IV.

###Suggestions
Start with clean and functional markup, and rely on the built-in Bootstrap components to the greatest degree possible before adding in bells and whistles. There's a lot of work ahead. The Bootstrap carousel may work for your image galleries, and swipebox.js is also another option. You may need to search out another 3rd party plugin that will give you the display options you're looking for.

The interactive chart is going to be cool. I was thinking of Chartist.js off the top of my head - but I'm interested to see what you've found in your research as well.

I'm wondering if a direct download link for your resume is the best UX for your mobile users. Mobile users may not want to download that .pdf without warning. Perhaps the resume link should direct to a resume page with a download option? Something to think about. Also, is there a contact form planned?

Great job - looking forward to seeing this take shape!

Links:
- http://brutaldesign.github.io/swipebox/
- http://getbootstrap.com/javascript/#carousel
- https://gionkunz.github.io/chartist-js/


I think you can use Build.VERSION_CODES.ICE_CREAM_SANDWICH instead (just for readbility)


Thanks for doing this, could you provide a screenshot of how this looks? I'm not a latex user myself.


30411/Alb V. Corina Bianca/Assignment 7.1 - Hash Tables - Open Addressing/Hash Tables - Open Addressing/hashTable.h

instead of
```
const float INITIAL_HT_SIZE_FACTOR = .25;
const float MAX_FILL_FACTOR = .80;
```

write
```
static const float INITIAL_HT_SIZE_FACTOR = .25;
static const float MAX_FILL_FACTOR = .80;
```


Line 728 ensures the indexed entry exists.


Nope, this excludes the file bin not the folder bin/


This is so much better.


ok, the second null should be a false? am i right?


This is looking great!


we are not making any server call here. progress needs to show only for network  calls


Tried it, works :)


for reference: here is the original file: https://github.com/intellimate/Izou/commit/19980cd773aedae4bb8d6c0a5d10217f8792b6f9#diff-02e4022fa858dbb52a88276f1b14b9a0R87


initial commit


How can I draw the entire texture (or lower half) white? The light color default should be white, but with the current code everything is black and we only draw the triangles over the black texture.


wait where did this go in a loop? and I used -1 0 1 to indicate the type of error that occured


I commented it too


Ended up doing the above in mktime_utc(). See commit 051aa5168a3374c3b00f60bc1e7d6d791fcfdb07.


Yes I wrote it.


New Exploit-DB site went live!


Ye i planned to move all cm_strings into slimmod_strings after fixing/adding all the stuff ;)


Added POSIX definition in 051aa5168a3374c3b00f60bc1e7d6d791fcfdb07.


No, just a regression test to ensure that existing URI versioning works with or without new page limiting.


Looks promising.  I am getting ready to travel and I will be back on the 30th.  But you can send that as a PR and start a discussion with @birdage and hopefully our BDFL @wrobstory.  :tada:


Maybe it should, but it didn't before and I don't know why.


Hello, thank you for reviewing and reading my code :+1: firstly.

There is no need for this kind of behaviour actually. What would be nice to do is to make it's lifetime configurable per request, session or web app lifetime. I forced it like this because there was no need for fast processing so I rather decided to have this object recreated completely and hooked with all the events per request.

At the time of writing this I was not taking any care for dependency injection or IoC, so maybe in the next writeup some day I will be adding more decupled code with managable lifecycle of the broker instance.

V.


Ok


Should be "4.02": `typeconv_112_01_01` does not build with 4.01 nor 4.00.


Same comment as above. You may want to use the attribute `typeconv_109_60_01` somewhere.


Can you do it now as we have found it?


I'm a bit worried about the ```androidboot.selinux=permissive``` part - I assume that's only used when booting to TWRP recovery but not active during normal operation?


On line 336, I think you must had something.


sample for liferay


I'll put up a PR soon.

On Thu, May 14, 2015 at 10:58 AM, Mark Schaake <notifications@github.com>
wrote:

> SingleInstanceUnitSpec
>
> +1
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/allenai/common/commit/11c355a756c18bd0bff90c30c9179833dc09dbd9#commitcomment-11200051>
> .
>


theano/sandbox/cuda/tests/test_opt.py:test_opt_gpujoin_joinvectors_negativeaxes

In the past, we where giving error only at function execution and it was a ValueError. Now this new detection is done when we build the graph and don't give the same error.


I don't mind if it falls back to the python code in that case.  But I can make it an assert.


Why?


salah tuh yoo


лучше писать так:

```javascript
if( exp ) {
  // code
} esle if ( exp ) {
  //code
} else {
  //code
}
```


yang bener ari keren


No translation on the static text at the end of this line. I'd add that for consistency.


а зачем через `$window` ?


@hzeller I think Rubin sent another mail w/ an updated code `MAKER#` ?


I see, checks so long as the notebook is installed.


yeah...but the type of the first function is void -> Boolean and the second is Boolean -> void


Why create this indirection? Why not put this in `__call__`?


i guess some sort of classcastException would occur


Hi .. how can i contact you ... i have a problem with an arduino code you've uploaded


Yup - checks if it can, otherwise not. In the future, I think this should be changed somewhat, but I did the minimum to get it running without notebook installed.


Вынести можно, но тут проблема, что в этом файле ("Auth.php") , проверка: if (!$uid) { throw ... } всего в двух местах и выводит разные сообщения об ошибке, так что думаю при таких объемах использования не сильно изменит картину.


* tagged ppl vs post likes
* tag frequency vs like amounts per tag


Fixed Rename to not allow only whitespace and implemented new.


On 14 May 2015 at 11:15, patrickod <notifications@github.com> wrote:

> @hzeller <https://github.com/hzeller> I think Rubin sent another mail w/
> an updated code MAKER# ?
>
The '# is finishing the input, it is not part of the code.
But strlen("MAKER") == 5 is the problem.

-h


`herelib` is enough as an attribute name. It is already part of an `ocamlPackages` attribute set, so we know it is some OCaml stuff.


teste


r+


ctivatorModel.call(); returned false, to indicate that it did not want to get called again. So ```.thenAccept(restart -> {```gets called, the first if clause is:
 ```if (restart != null && restart.equals(0)) {```
this should get triggered (as the debug message said: ```debug("Activator: " + activatorModel.getID() + " returned false, will not restart");````, but restart cannot equal 0, because 0 is an integer and restart is a Boolean, the second is also impossible, so it goes to the else-clause (the restart-clause)


Sure, I just think my version is more pythonic; just a matter of taste :p


You can relax this constraint to "4.00".


saw approval from MonsieurNicolas
at https://github.com/stellar/stellar-core/commit/42f187626d4cfa1def6bf242a1411f1af8279921


merging stellar/stellar-core/change-trust-test = 42f18762 into auto


stellar/stellar-core/change-trust-test = 42f18762 merged ok, testing candidate = 0fc481e5


Creature* /*creature*/
next method the same


So, this here switch is a nested copy switching on the same thing as the outer switch, right? I've no doubt it's doing the right thing, but I can't help but think it's not very readable. How about doing as we do for `isArray` and `arraySize`, and setting `interpretation` above before falling through?


Yes, this is needed for the GPU  GpuElemwise uses the c_code of the scalar ops to generate its c_code.  So this can be indirectly used on the GPU with float16.

I can't tag it _f16_ok since it will not work correctly if used on its own for float16 inputs.

I also see that it need an adjustment of the threshold for float16.


I found out that CREATURE_EXTRA_FLAG_INVISIBLE (creature_template, defined in creature.h, used just once in in creature.cpp) makes trigger NPCs invisible. But it did not work in my case for some reason (and I tested this dbscript several times). Well, in any case it is not DB issue.


Nice!


@andrewrk I have to say I am a bit confused. Why go all the trouble of starting a new thread instead of just calling uv_async_send on the GetBuffer method? AFAICT uv_async_send can be called from any thread.


Well done!

Btw, could you provide the feature like this? XD
![2015-05-15 2 21 28](https://cloud.githubusercontent.com/assets/6128538/7638720/4919c7e6-faa9-11e4-9a21-6b84608d1616.png)


spell-check, please!


bah, silly me. I didn't read the context!
On Thu, May 14, 2015 at 11:18:03AM -0700, Henner Zeller wrote:
> On 14 May 2015 at 11:15, patrickod <notifications@github.com> wrote:
>
> > @hzeller <https://github.com/hzeller> I think Rubin sent another mail w/
> > an updated code MAKER# ?
> >
> The '# is finishing the input, it is not part of the code.
> But strlen("MAKER") == 5 is the problem.
>
> -h
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/hzeller/rfid-access-control/commit/763a7e1dbba2c65d059ee2a003c8c27ed46563fd#commitcomment-11200481

--
Patrick O'Doherty
+1 (650) 701-7829


I didn't test it but something like this should work:

```
glClearColor(1.0f, 1.0f, 1.0f, 1.0f);
glClear(GL_COLOR_BUFFER_BIT);
```

To only operate on one half of the texture, use `glScissor()` to specify a rectangle that you want cleared, and enable the scissor test with `glEnable()`.


Also need to update path to Zone:
```Shel
'node_modules/angular2/node_modules/zone.js/dist/zone.js',
'node_modules/angular2/node_modules/zone.js/dist/long-stack-trace-zone.js',
```


Depending on how people feel about my previous comment, I suggest we do the same thing here, rather than have a nested switch.


I don't see how that is related to this commit. Can you open a separate issue?


Hi Steve, sorry about that, we use the google location APIs to locate the jobs and it wasn't able to be accurate enough with just UK.  I've updated the file to have Location set to '70 High View Avenue, Essex, UK' and it's now live.  You can view the job here: http://pythonjobs.github.io/jobs/compoundpartners.html

Many Thanks

Steve


I see the point.
However, this will not be present in soledad.server 0.6.5
... unless we re-tag (ugh) or we cherrry-pick this into the debian package.


remove second lcIntro here. Should be a single lcIntro in the topic.


Да, я понимаю что можно было использовать полиморфизм. Но я не понимаю каким образом сделать разбивку по типам. Даже если каждая фигура будет печатать по-своему, неизвестно в каком порядке они расположены в списке. Или я неправильно понял задание?


Metadata/Monsters/Monkeys/MonkeyBossInvasion1	Blood Stasis
Metadata/Monsters/Monkeys/MonkeyBossInvasion2	Blood Morpher


let's leave as many semantic changes as out of scope as possible;

Since we don't have any tests for these branches it would be good to keep them as warns


No, not the big bug of "it doesn't fecking work".


Incorrect. On SickGear it is still located in the .sickgear folder, not in the runscripts folder. Only SickRage has a runscripts folder and both SickBeard and SickGear don't (yet).


Missing spaces inside parentheses and between the closing parenthesis and the brace at the end of this line. See: https://make.wordpress.org/core/handbook/coding-standards/php/#space-usage


Is there a specific reason for mode `0600` instead of `0644`?  It's not secret data right?


Missing a space before the closing parenthesis at the end of this line.


Missing spaces inside parentheses and before the opening brace


Bravo prima o poi (magari quando togli anche le robe di bootstrap che non servono) tutti gli style= vanno sostituiti con i css.

Magari convertire anche i px per le dimensioni in em (se non sbaglio, ma non ci giuro, i px sono i pixel appunto mentre gli em dipendono dalla grandezza dei font che imposta l'utente


This is not output escaped. Also missing spaces inside parentheses.


Missing spaces inside parentheses.


I created a new branch 'finger' in this repo. That way my patches will not get in the way of yours and you can cherry-pick them from the new branch. Let me know if anything does not commit that way.


Thanks man... Could you please answer my hangout?


Missing spaces inside parentheses. Should be using a yoda conditional.


Come predefinito, quindi se uno ci vede poco c'è li ha più grandi)


Static text should be using `esc_html_e` to be consistent with the rest of the plugin.


could you use `isdefined` here?


what


I should have rethought the BabelJS "bjs" abbreviation...


The thread is to wait for `groove_encoder_buffer_peek` to return. It blocks until there is a buffer in the queue available. The problem that you discovered is that peek was returning 1 (and not blocking) because once a buffer was available on the queue, peek always returned immediately until the user called `getBuffer` enough times to empty the queue.

So now when the 'buffer' event is emitted, a flag is set to so that the thread waits on the condition until the user calls `getBuffer`, at which point the thread is woken up and the flag cleared so that another 'buffer' event is possible.


I know, that's why I backed out this change.


there is no need for multiplication by 10 twice and finally dividing by 100.  Just multiply all first and divide ...

New equation also inverted meaning of D8[], now it sets minimum leveling instead ...


Awesome! :+1: :boom:


То есть в списке инициализации будут вызваны 2 конструктора, которые по сути делают одно и то же?


A new line here.


Hi Eddie,

I'm watching you use git, and it seems to me like you are using it in the most complicated way possible. You may want to review procedures. You should not normally need to use 'git merge', except in exceptional situations.


Missing spaces and yoda conditional. Should be formatted like:
`if ( 'conditional' == experiment.activation_mode ) {`


`table`'s type here should be `*gorp.TableMap`, I think.  Same with everywhere else you use `table`


but restart is not a boolean.. it was an integer.. It always worked for me


You need to import the modules/classes/functions you are using in any file.


Missing spaces inside parentheses


Missing some spaces and also should be using yoda conditionals.


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


@michaelwehrley do you know what `>>` does, and did you intend to use it here?


This approach is basically how I implemented the eventually-consistent approach in the `reactor-data` module.

IMO it's more flexible to have every mutation to the session implemented as a delta. If you want to update attributes immediately and never queue them, we can just override the `addDelta` method and immediately issue an update rather than lazily applying mutations.

I wanted to catch every update to an attribute rather than only the last one for a key, assuming that all updates to a session might be important (they might mutate a complex value) so I didn't want to maintain a separate map of "to update, change these".

I was hoping we could start moving a little towards a shared, non-servlet use case, but if you just want to stick to the thread-bound model for the time being then that's fine. It just means we'll have to rewrite it when we want to use it with the async web stuff that Rossen, Stephane, and I are working on. That stuff is 180 degrees different from the thread-per-session model.


All glory to you !


Should be `echo esc_url( get_full_permalink() )`


Enables the automatic adornment of code blocks by the `\smaller` command/environment provided by the LaTeX `relsize` package (cf. <http://www.ctan.org/tex-archive/macros/latex/contrib/relsize>).

Only the `CODE` and `VERBATIM*` cases in `latex.c` have been altered. The respective output is now preceded by `\\ifx\\smaller\\undefined \\relax \\else <applied relsize macro> \\fi`. By embedding the `relsize` commands in a conditional TeX block, the availability of the `relsize` commands is reassured.


For the Reactions.tsv SOLR table, the source file is missing the is_transport, deltag, and deltagerr fields.

I don't understand the code field.  It seems to be a duplicate of the equation field.

I don't understand the abstract_reaction field.  In the current SOLR tables there are no reactions that have  a non-null value.

We don't have any reaction aliases stored in the Aliases directory.  Currently the aliases files have fields for both default and plantdefault IDs.  We'll need to change that to just the IDs from the corresponding master file.  And we need reaction aliases to build the pathways and ec_numbers field.

And we should be storing the reaction equation in a compartment free notation that can then be used to build the equation, definition, stoichiometry, and compounds fields.


I differentiated there between a security denial (0), a crash (1), and a permission denial(-1)


ignoring `E501` negates the `max-line-length` setting below.


Please open an issue in the repository for the code you're having a problem with, and we'll do our best to help you out.


WordPress VIP does not allow usage of `$_REQUEST`. It must be explicitly `$_POST` or `$_GET`.

See: https://vip.wordpress.com/documentation/code-review-what-we-look-for/#using-_request


See above


Usage of `$_REQUEST` again. See above.


can we give it a more specific name, like cacti_key?


In my first commit I have just given intro of github .. a very brief one.


Sorry please discard above comment.


can we also clear the map at the end of the function since it's not being used anymore?


Getting this error when trying to compile with VS2008.
```
1>.\HLEMain.cpp(444) : error C2036: 'const void *' : unknown size
1>.\HLEMain.cpp(444) : error C2440: '=' : cannot convert from 'const void' to 'i16'
1>        Expressions of type void cannot be converted to other types
1>.\HLEMain.cpp(446) : error C2036: 'void *' : unknown size
1>.\HLEMain.cpp(446) : error C2120: 'void' illegal with all types
```


curious why we're suppressing an exception here? isn't that a case where there's a legitimate problem?


I believe this is a mis-pasted typo. Will change back in a new commit.


Duplicating the code between the two `fileSaveToFile()` methods doesn't seem ideal, and it's a little difficult to follow what's going on with the #ifdefs. What do you think to approaching it like this?

- Unconditionally define all the virtual methods (both the Houdini 13 and 14 ones), just documenting which is required for which version with a comment.
- Put all the code that does the work in the Houdini 14 versions.
- Implement the Houdini 13 versions in terms of calls to the versions for 14.

I'm not sure how I feel about the "unconditionally define all the virtual methods" suggestion, but I definitely think some solution involving no code duplication would be preferable.


I think I do.  It takes the response that I get from the url, and it copies that response into the specified file.


might be better to make it APPC_NPM_VERSION.


fast-forwarding master to auto = 0fc481e5


Read this: http://bogojoker.com/unix/skills/append.html


@pavlm Потому что теперь ссылка для подтверждения отправляется с помощью sendWelcomeMessage. Если вы переопределяли представление welcome, то вам нужно обновить его в соответствии со сделанными в модуле изменениями.


Not very happy with having that function in `deploy.go`, should I create a `utils.go` for it?


oq é teste?


Since you have converted it into a switch statement, I would add a default: just to be on the safe side.


Since above we have FFmpeg libraries, I presume this refers to DVD playback component, right?

Can we just not name this as such?


Did you test this? It is breaking for me


We already know we we're working on an async web approach that will definitely require shared access to a session. It's a fair argument to say that that stuff doesn't exist yet so we'll defer to address this until it does. I'll do it however you want. But sooner or later we'll have to address the shared, non-servlet use case and I suspect that anything we'll want to show at SpringOne would benefit from session support.

IMO we shouldn't force components in core to be accessed from a single thread simply because we know we already have use cases that don't fit that mold and more are coming as we flesh out the async web support.

But I'll do it however you want.


quem eh rafa?


Ricardo Carvalho


Good call!


retest this please


Shouldn't this be `if not team`?


Is it possible to just define a typedef out of the way somewhere, so we can get these #ifdefs out of the meat of the code?

```
// do the conditional bit once, out of the way
#if UT_MAJOR_VERSION_INT >= 14
typedef GA_PrimitiveGroup GroupType;
#else
typedef GA_PrimitiveGroup GroupType;
#endif
// use the typedef in the code itself, to keep it readable
for( GA_GroupTable::iterator<GroupType> it=geo->primitiveGroups().beginTraverse(); !it.atEnd(); ++it )
{
}
```


@da-anda @jjd-uk
I finally realized that this is AudioEngine related and that excludes this from "teh" Capitalization. Right?
I wanted an excuse to remove more unnecessary quotations.

So if this is right we should leave it like that and only one left is the Video component, see below,,,


Is this configuration still relevant? I believe we already have enabled and disabled compressor list configurations.
In fact, I think it's a good idea to keep compressor explicitly set in the conf file. It can actually serve as a white list to replace the disabled configuration; and it can serve as a plugin point to specify the jar files to look for, which enable use to decouple client configuration and compressor package setup.


That's bashism, with your shebang being /bin/sh. It will likely break for a lot of people.


There is no need to strait way  remove the listener, this is already done if the widget is in the rurrent scene.


> Unfortunately this discussion did not catch any momentum. But I think it's pretty much what you're proposing.

A chat is probably not a good platform for discussing topics of this scope. :)

Using a `flow_controller` is closer to the design I would prefer.

> > I'm thinking about actor-framework/actor-framework#227 for a while now and this "fall though" is something I consider harmful.
>
> Why?

A feature like this only makes sense to build inheritance-like behavior definitions where you wiretap or filter certain messages in the superclass. This design is simply too brittle for my taste. One `become` with an "unaugmented" behavior and your "abstraction" unfolds (see commit message above). Having multiple levels of abstractions inside a single actor makes code hard to reason about. Instead of layering message processing steps in a single actor, I would always prefer using multiple actors instead with a forwarding chain.

I don't buy the "local decisions" argument, btw. Take the middleman actor for example: it provides an interface for publishing local actors and gaining references to local actors. It's implementation, however, is simply to establish the initial TCP connection if needed and forwarding all calls transparently to the `basp_broker`. It uses response promises to keep track of original requests. Any client of this actor will *never* receive a message from the `basp_broker` and the `basp_broker` is completely hidden away.

You don't even need to keep track of the exact number of messages in a mailbox: you know how much pendings requests the worker has at the time of receiving a new task. As long as the forwarding actor does nothing but bookkeeping, its message handlers are fast enough for ignoring the "actual" situation in the worker's mailbox. If you have 20 pending tasks in your list with an average (measured) runtime of, say, 50ms each, you don't need to know if there is one response in your mailbox. It does not matter do detect an overload situation where you receive new work faster than your worker can handle.

From a complexity point of view this is a huge difference. Each actor itself is simple. Easy to understand and to reason about, while the composition is *also* much easier to understand. You simply add another actor to your chain. `logger <-> flow_controller <-> worker`. Instead of complex "augmentation" of message handlers, I would simply compose actors. Instead of `spawn<worker>`, you do `spawn<logger>(spawn<flow_controller>(spawn<worker>()))` which you then can flatten using a `vast::spawn_augmented` for example.

If you are afraid of a few extra cycles: go measure it. I'm sure the overhead is not an issue. :)

It's probably a good idea to spawn the flow controller detached, btw. If all threads of the scheduler are busy, the `flow_controller` will sit and wait in some queue. Detaching it means that it will always have a reasonable response time. Another benefit and something you can't do with your approach unless you detach each worker. Using a single `flow_controller` for multiple workers is also simple to do with this approach (and simple to change afterwards!).


https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards and configuring PHP CodeSniffer for the team's editor should take care of it. Based on @bcampeau comments I'm not sure if you guys know this already or not, so thought I'd throw it out there. Cheers, keep up the good work here.


Yeah, wasn't causing any problems after I merged it.


In your context diagram the mail that one receives when he's being added to a circle, is missing.


Please add 7.0 as well.

```yml
  - 7.0
  - hhvm

matrix:
  fast_finish: true
  allow_failures:
    - php: 7.0
```


Just for peformance, to avoid add or remove the widget that is already there.
Not having this line wont add one listener twice.


Line 385 is the problem grading/execute.cpp:385:3: error:
      use of undeclared identifier 'my_char_args'


A small error.


Feels like the judicious use of a typedef up top could avoid all of this conditional compilation?


Please add `- 5.4.7` above `- 5.4` since that is our lowest supported version.


For the archives, I've defined this Capistrano task by now:

```ruby
namespace :deploy do
  after :normalize_assets, :gzip_assets do
    on release_roles(fetch(:assets_roles)) do
      assets_path = release_path.join('public', fetch(:assets_prefix))
      within assets_path do
        execute :find, ". \\( -name '*.js' -o -name '*.css' \\) -print0 | xargs -0 gzip --keep --best --quiet --force"
      end
    end
  end
end
```

Not battle tested, but seems to be correct.

@josh if you'd consider taking this back with a flag (default to false if you want), I'd volunteer of course.


I have been running grunt and made sure that passes. But guess this spaces v/s tabs issue. will fix it


Please specify a version number.


In older versions of _semver_, an exception would be thrown for ```.parse()``` is an argument was null.


To make our change minimal possible I will remove this (and such) change(s). I believe the hive collaborators will take care of such changes themselves.


We do, get_path will return None if no device was found, and I use this to detect whether or not we should display brightness management widgets,


previous post has been updated. but edb29139ddafe0f020cf21e3ade323b0b5074730 fixes the minor issue I raised earlier.


Change this to class map for now. Otherwise the syntax tests will fail.


Not sure I get you.


`@return void`


Please fix the indentation.


Blank lines are not inherently disallowed, but blank lines are most often
used to show division of a task into smaller parts, which should be done
with the use of different functions.


@da-anda @jjd-uk

As per earlier review this was a good generic message that could replace all "Check the Log" and "See the log" inconsistent entries.

They all now been added like that or small variation where user is advised to check settings or wait a while or check log. The Line break make this more presentable (no wrapping) in the box, more easily readable and  makes the message more in line with how you see message boxes on other software.

I also added the "[CR]Check the log for more information about this message." to where I tested these entries in skin and they were missing.


I have to confess to being out of my depth and not understanding all this GU and GEO stuff, but I don't see any methods being moved - are they actually being copied from one to the other rather than moved?


Do we need to use both `dirname` and `__DIR__`?


reference pull #1 by @kferris10 (thanks)


in decodeFromMemcache, you're dealing with the values from `json.Unmarshal`.  Assuming that the values from memcache are loaded into a `interface{}` type (which they probably should be), we know that reflect.Struct *cannot* be part of the data from memcache, because json only creates types:

1. `[]interface{}` to represent arrays
2. `map[string]interface{}` to represent objects
3. `float64` to represent numerical data
4. `bool` to represent booleans
5. `string` to represent strings
6. `interface{}` to represent null

As such, I think the code should look something like:
```go
switch src := data.(type) {
case []interface{}:
    for _, v := range src {
        reflect.Append(targetType, reflect.ValueOf(decodeFromMemcache(v, targetType.Elem(), table)))
    }
case map[string]interface{}:
    for k, v := range src {
        col := table.ColMap(k)
        if col == nil {
            // return an error, probably, since the target type does not have a field to apply this value to
        }
        targetType.FieldByIndex(col.FieldIndex()).Set(reflect.ValueOf(v))
    }
default:
    // bool, float64, string, or nil
}
```


Yeah we need to change it again now, since it's 1.0.1 now, properly.


Yes, using just `__DIR__` will try to load vendor from the `tests` dir.


Ah I see. Honestly I didn't look at the query when asking the question. You are right, code can be improved. Check out query builder function for ideas what to do. Keep in mind that  $active = TRUE can't be included as query condition but used to determine when condition status = 1 needs to be added to the query.

Next time instead of saying HACK you may want to add @todo comment explaining what would you like to do.


Haven't fixed these yet, but keeping notes about what I'm finding merging master into this branch ...

The new [`mdn-youtube` ckeditor plugin pull request](https://github.com/mozilla/kuma/pull/3210/files) moved:
`media/js/libs/ckeditor/source/plugins/{youtube → mdn-youtube}/`

The new [`fontfaceobserver` library from the lazyload font pull request](https://github.com/mozilla/kuma/pull/3192/files) was added under `media/js`, along with a new `media/js/fonts.js`.

The related updates need to be made to `settings.py`.


Please change this to classmap instead.

```php
"classmap": [
		"tests"
],
```


I'm getting an error when running "php artisan"

"Cannot redeclare class Event"

When i remove this line, it works.


Thanks nig nog, I just gotta change the time to 5:00 pm and then I think we oughta be alright. Let's get a beer soon and jam.


This reverses the accidental reverse of the commit before it. Test still doesn't pass because of database problems.


Is this because the defaults changed between Houdini versions?


Hello, in my new GPD Q9 Sonic Adventure 2 crashes all the time, what build of Reicast can I install to solve this?


Aah okay, I see what you mean. Maybe just use `..` instead? Then we will skip one function call.


Hehe ya I fixed that but haven't pushed yet. :P but even with the screwed up one I have no heat issues :(


This is an issue disabling SSE2.  I should have tested further.  You can fix this by type casting.
```
void swap_elements(void * vd, const void * vs)
{
#ifdef SSE2_SUPPORT
    __m128i RSP_as_XMM;

    RSP_as_XMM = _mm_loadu_si128((__m128i *)vs);
    RSP_as_XMM = _mm_shufflehi_epi16(RSP_as_XMM, _MM_SHUFFLE(2, 3, 0, 1));
    RSP_as_XMM = _mm_shufflelo_epi16(RSP_as_XMM, _MM_SHUFFLE(2, 3, 0, 1));
    _mm_storeu_si128((__m128i *)vd, RSP_as_XMM);
#else
    i16 temp_vector[8];
    register size_t i;

    for (i = 0; i < 8; i++)
        temp_vector[i] = ((i16 *)vs)[i ^ 1];
    for (i = 0; i < 8; i++)
        ((i16 *)vd)[i] = temp_vector[i];
#endif
}
```
I will let cxd4 apply the fix since this may not be what he actually envisioned.


Instead of copying it, it appends it.  We're on exact same page now.


All the occurrences of these line breaks should be removed from the html. Using <br> tags for padding is bad practice.

The correct way to do this is to adding margin and padding through css.


Changed to  `__DIR__ . '/../` instead of `dirname(__DIR__)`


Looks like I mixed up my commit names....


Этот метод и метод выше используются только в классе `RDOPlotView` и только для удаления записей. Думаю, стоит вместо него сделать метод `removeIndexFromMaps(AbstractIndex index)`, который будет содержать в себе проверки на наличие индекса в мапах и удаление его оттуда (в общем все то, что сейчас делается в `RDOPlotView`), а из `RDOPlotView` его просто вызывать.


Лучше переименовать во что-нибудь вроде `lastUnreadEntry`, ну или в таком духе.


:-D
OK pause a minute. let me catch up.
the chrome store and standalones are now 1.0.1, so the GitHub source
(VERSION, MANIFEST* which were changed to 1.1 after publishing) needs to go
back to 1.0.1 ?

  _____

From: SchizoDuckie [mailto:notifications@github.com]
Sent: Friday, 15 May 2015 06:52
To: SchizoDuckie/DuckieTV
Cc: garfield69
Subject: Re: [DuckieTV] annoying webstore versioning doesnt match semver...
(e4732b9)

Yeah we need to change it again now, since it's 1.0.1 now, properly.

-
Reply to this email directly or view
<https://github.com/SchizoDuckie/DuckieTV/commit/e4732b9ff6de8e54fa8a2ce4e1b
485b873f582ae#commitcomment-11201180> it on GitHub.
<https://github.com/notifications/beacon/AGnK-CylhaQUA8_tjG6Vv5HN3QYrL-vNks5
oJOZUgaJpZM4EatEX.gif>

---
This email has been checked for viruses by Avast antivirus software.
http://www.avast.com


@da-anda @jjd-uk
I finally realized that this is AudioEngine related and that excludes this from "teh" Capitalization. Right?
I wanted an excuse to remove more unnecessary quotations.

So if this is right we should leave it like that and only one left is the Video component, see below,,,


Yeah ill remove the mode. That was just remnants of my own tests.


Are the changes here because we're comparing a new generated-in-memory torus from Houdini 14 with an old generated-in-the-past-and-saved-for-later torus from Houdini 13?


Pizdatiy commit!


@da-anda @jjd-uk
Since above we have FFmpeg libraries, I presume this refers to DVD playback component, right?

Can we just not name this as such? So I can get rid of those quotes that way...


Any idea why enabling texturing on Linux causes errors? This seems worrying.


@da-anda @jjd-uk

As per earlier review this was a good generic message that could replace all "Check the Log" and "See the log" inconsistent entries.

They all now been added like that or small variation where user is advised to check settings or wait a while or check log. The Line break make this more presentable (no wrapping) in the box, more easily readable and makes the message more in line with how you see message boxes on other software.

I also added the "[CR]Check the log for more information about this message." to where I tested these entries in skin and they were missing.


what?

On Thu, May 14, 2015 at 1:39 PM, adrian-the-git <notifications@github.com>
wrote:

> Awesome--so glad prose is working for you!
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/BeautifulTrouble/beautifulrising/commit/8be00f0e7e53a064fe1a7f7f3f854eb9043d412e#commitcomment-11199731>
> .
>

--
Andrew Boyd
347/228-7416 (cell)
andrewontheroadagain (skype)
andrewontheroad@gmail.com (email)
http://andrewboyd.com
+++

"Last year, we refused to keep losing. This year, we teach ourselves to
win." --Max Haiven
"Everything's coming together while everything's falling apart." --Jamie Henn


Thanks, Hope your kernel success! Cheers!


@da-anda @jjd-uk

As per earlier review this was a good generic message that could replace all "Check the Log" and "See the log" inconsistent entries.

Added the "[CR]Check the log for more information about this message." to where I tested these entries in skin and they were missing.

The Line break make this more presentable (no wrapping) in the box, more easily readable and makes the message more in line with how you see message boxes on other software.

Same below on next 3 entries


Может там другой коментарий по длинней?

2015-05-13 12:33 GMT+02:00 Maxim <notifications@github.com>:

> Не к спеху!
>
> On 13 May 2015 at 18:12, Markolm <notifications@github.com> wrote:
>
> > Странно, после школы посмотрю.
> > On 13 May 2015 05:23, "Maxim" <notifications@github.com> wrote:
> >
> > > @Markolm <https://github.com/Markolm> работет!
> > > чуть чуть с whitespace неровно только:
> > > [image: image]
> > > <
> >
> https://cloud.githubusercontent.com/assets/10990645/7603258/0da45c38-f984-11e4-83e4-901ccb80e0c1.png
> >
> >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/maxim47/C-enums/commit/9a77151170039b68dccc19fdfa983a762e679c19#commitcomment-11168039
> >
> >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/maxim47/C-enums/commit/9a77151170039b68dccc19fdfa983a762e679c19#commitcomment-11169341
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/maxim47/C-enums/commit/9a77151170039b68dccc19fdfa983a762e679c19#commitcomment-11172884>
> .
>


Something we can talk about:
I feel that if we put the enColumn field into the TColumn union, it will result in a cleaner and better-organized interface. Because TEnColumn is conceptually still column data, and TColumn is where all kinds of column data goes (TBoolColumn, TStringColumn, TBinaryColumn...). This may also reduce data overhead since we actually put the columns filed into use when we have only encode data.


Might trap unaligned stores, but OK.


Согласен )


One more place to change to a list of pointers :-)


I'll investigate further, thanks for the feedback guys (=


As per earlier review this was a good generic message that could replace all "Check the Log" and "See the log" inconsistent entries.

The line break works best hereas does form all messages of this type for same reasons so I wont repeat them anymore. Hope this is ok it really does work better if you test.


You should be submitting pull requests, not pushing to develop.  Additionally, we don't want a dummy button in there - it would just confuse a user.  You should instead keep this on a feature branch until it is ready


The size attribute is supposed to represent the length of characters of the input. To change the visual size of the box, using the css-style width attribute is more appropriate. The general rule is that if it doesn't effect the functionality or organization of the content (design) it should be done in CSS.


yep ninja'd

btw i think vs2008 solution probably should have SSE2 set in the project settings


Not that much, omniarmv6 is for omnirom and androidarmv6 is for cyanogenmod

Here is the download link
https://www.mediafire.com/?tovwq8ojvt054qt


DONE

That's embarrassing ;( The error was actually different due the other change. Fixed now.


see #3


nope


ACK

Yes, let's discuss this offline.


@participant already set in BaseController


:+1:  thk brother


It's deprecated as of GLES2.0.

https://code.google.com/p/angleproject/issues/detail?id=313


Seriously?


Здравствуйте,
необходимо добавить:
curl_setopt($ch, CURLOPT_SSL_VERIFYHOST, 0);
curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, 0);

т.к. не везде курл подгружает системные сертификаты, у меня возникла такая проблема, ошибок не выдавал, пришлось мучится и отлавливать...


Te comento que si hay alguna cosa Edraw Max, que es parecido a lo que mencionó José no se si es correcto pero es lo que pude bajar.


.html() is different from .replaceWith() . Use in different cases carefully. in this case, html will incur double <div> with 'activity-id'


@XileForce the methods cpus_intersect and cpumask_intersects actually look the same, except one is using the cpumask bits and the first one is using nr_cpus


I used `buckets[i].empty || buckets[i].deleted` because the `HASH_FILLED_MARK` literal is huge (highest bit => 8-byte code constant).


Uhhhh.... yes lol, thanks.


@da-anda @jjd-uk

Line breaks here help make this more readable, since this is both scrolling and wrapping, like this on the extra long messages you dont go back to top when getting lost just pick up the line you were at if it happens, everyone in the house agrees this works better.


Is that bad? I just switched to what you were using.


Why the list()?


I still get the errors running `collectstatic` on this commit. Was there a `cssmin` step we need in puppet somewhere?


Isn't android the only platform using GLES at the moment?


@robpaveza . I think you are referring to ApplicationException guideline which was around .net framework 1.1. Catching `System.Exception` is still a bad practice: https://msdn.microsoft.com/en-us/library/vstudio/ms229005(v=vs.100).aspx.

@sgrebnov  - As for better error message - IMO catching `System.Exception` and putting an error message like "download failed" is not very useful. I'll let Alex make the final call here.

`async` here causes more complexity in how `download` code should be written as it pertains to handling static state like `InProcDownloads`. My point is that `async` here does not buy us more scalability or UI responsiveness and can complicate the control flow and causing exception handling of the caller to not work. If we decide to make it async - we will have to ensure we design correctly for it and it requires some careful thought especially around best practices of void async methods (as opposed to ones returning Task<T>) and on exception control flow. Synchronous plugin methods are easier to work with here.


I think this hunk will make compile fail since the lists within this ifdef it tries to free (types_nua, deptree) was made local with commit 7716bf31de5030b761613834e11e4e62f36403a5.


This seems like dead code. `cs` will never be non-null here if you follow the code above.


This returns type name without package.


ditto.


I don't agree with this one, see https://github.com/D-Programming-Language/druntime/pull/1229#issuecomment-98566967. This optimizes the AA implementation for one of our microbenchmarks, but produces __much__ worse results for the rest (2 cache misses for big AAs).


Wouldn't this set the flag twice if `h.__monitor` is already set? Seems like a waste, what about a flag saying it was created?


There has been an open question on this as to whether we should store the time and allow a retry later if the v3 source is down.


Same thing, seems like dead code.


I'm using it right now on desktop linux.


affinity group will be there for vnet which are based on it. Even though it is highly recommended to use location based vnets, old ones are still there based on affinity group


Fixed in #89


Shouldn't the core engine be a <file> with the adapter like in NUnit3VisualStudioTestAdapter.nuspec, not a dependency? If it is a dependency, it will get copied as a reference into the project which is not what we want.


Fixed in #90


To continue the previous point: this bitmap field can be removed as well when we put TEnColumn inside TColumn. A null check of the TEnColumn field will tell us whether there is encoded data; although the performance may not be great, it results in simpler interface. If we want to improve the performance there could be other simpler ways instead of using a bitmap.


That might be worth a try, AFAIR dmd never reused the mask, but trying to save it in a local variable in findSlot wouldn't allocate a register for it.


What do you mean? I assumed that, as no other action was taken in the original way, that unknown ReceiveTypes should be ignored. What do you suggest I place in the default case?


Yes... It was NOT easy to figure that out. :D


This also seems odd that we are distributing the framework as binaries. These should be dependencies. Is this even needed?


Понял в чем проблема.
После регистрации я смотрел письмо в yii-дебаггере, там отображается текстовая часть письма, и в ней нет ссылки подтверждения, а в html части письма эта ссылка есть.


Let us consistent on 'not found' message

A virtual network with name "%s" not found


I made a mistake. PSMove.connect_by_id(i) returns a pointer (c_void_p) to a PSMove object. We don't actually want that. We want an instance of your custom PSMove class. So...

```Python
controllers = []
for i in xrange(count):
    controllers.append(PSMove(i))
    controllers[i].connect()
    controllers[i].tracker = tracker
```

And you'll have to edit the PSMove class so it takes an id [here](https://github.com/davidlu89/psmoveapi/blob/c9a8ec3ef6eefff06c557647baa5f527ec53e354/bindings/py_ctypes/psmove.py#L24-L25):
```Python
    def __init__(self, id=None):
        self.id = id
        self.version = 0x030001

    def connect(self):
        if self.id:
            return libctrl.psmove_connect_by_id(self.id)
        else:
            return libctrl.psmove_connect()
```


```js
return fs.writeFileAsync(tempSrc, code);
```


Ah! I didn't look at the type.


It's a gambiarra jejejeje


So you're saying [this](https://github.com/Thunderbots-5604/2014-Code/blob/master/api/movement.c) setMovement function should be split into 8 functions?


In V2 the core assemblies are  also copied in as references, but the install.ps1 scripts sets them all to Copy Local=false.  It would be better if the whole package was a solution package, but something prevents that working.   In V2 however, the core is included with the adapter, and not as an reference, and if it is now included as a reference it doesn't feel quite right for the adapter's install.ps1 to manipulate that package.


Removing the minimal validation means that an empty file is valid one. Don't think this is what we expect from this function.


Note this is a known issue: https://github.com/neovim/neovim/issues/2368


Sharpen your eyes https://github.com/MartinNowak/druntime/commit/026b69f8e102d9806751bfeac0a6b2654f865c9b#diff-d609308f6ce1b06db16e214b30db6f74R326.


##Milestone 3 Feedback
Excellent work Tim. I love your layout ideas, and the concise focus. You wireframes clearly communicate your layout ideas with no fuss. Your Milestone 3 passes at Tier IV.

###Suggestions
This should be pretty straightforward to develop. Start with clean and functional markup, and rely on the built-in Bootstrap components to the greatest degree possible. The simplicity of your layout should be fully responsive out of the box using the built-in Bootstrap Grid Classes to lay out your page.

Remember to keep file sizes in mind for large background imagery (< 500kb is ideal for desktop, and <100kb for mobile). Bootstrap has some built-in classes for image shapes, and Font Awesome is a great choice for additional iconography (links below). Overall - I think you've made some great choices here. Looking forward to seeing your project take shape!

Links:
- http://getbootstrap.com/css/#images
- http://fortawesome.github.io/Font-Awesome/


Obey the [comment](https://github.com/MartinNowak/druntime/commit/026b69f8e102d9806751bfeac0a6b2654f865c9b#diff-d609308f6ce1b06db16e214b30db6f74R309) above, this will only be called when the object doesn't yet have a monitor.


rax retry tests


You sure I should use the cumulative density function?

For the too-few blocks case:  I count 11 blocks.  I didn't see 11-or-fewer : I see 11. So it seems to me I should check against the "What are the chances of seeing only 11 blocks" threshold.

Next time I check, I might count fewer than 11; if there are fewer, then the alert will happen then.

Same rationale for too many blocks. If I count 111, I didn't count 111-or-more....

BUT: I'll defer to your superior knowledge of statistics if you're sure you're correct. I still regret not taking statistics in college when my brain wasn't full....


Ubuntu is kinda big for this. Maybe Debian stretch or one of those mini distros like Alpine?


@karissa sweet, thanks for adding this.

- I am not a fan of using `live` here, as it is used in commands like `dat pull` to mean a different thing. I'd rather just start with the convention of having the last argument be `-`, so `dat merge A B -`
- Maybe make `dat merge` take 3 arguments: `<headA> <headB> <file>`. So a workflow might be to do a `dat diff`, save it to a file (e.g. `diffAB.json`), work with that file manually using R or python, and then when you're ready to `dat merge A B diff.json` to apply it. Then if `<file`> is `-` we will just listen for STDIN.
- Just to clarify, if you specify a `merge-tool` then that means that we would be doing the equivalent of `dat diff A B | <merge-tool> | dat merge A B`, but without requiring the user to do all the piping, correct?


Thanks. I was not running the latest nvim from master btw.

NVIM 0.0.0-alpha+201505011649 (compiled May  2 2015 12:15:21)
Commit: 205466830207a920c62146b7b689fac2e395431a


Is there a way to have this plugin transform a query of '!wde fnord' to a query to wikipedia via duckduckgo?


`stable` hardcoded?


This check is already in server.js, so IMO no need to check it again here.


This is my require js working example


glGetError can be slow in some configurations, so perhaps this should be configurable or only operate this way for debug builds?


These lines are still needed for the upgrade from 2.0.


These two can be passed in as separate arguments instead of being split in here.


After chatting on IRC, I think I understand what is going on now. Maybe we should for Desktop Linux + GLES as one discrete chunk / PR?


How about inherit from std::runtime_error instead of std::exception?


@maxogden

* sounds good,
* i'm okay with that
* yeah, also if the merge tool uses stdin to take user input, doing it inside node allows the values to be piped to the merge function inside node.js


ups, seems like I messed something up there. Not a good idea to store files temporally on a FAT hard drive..


@antonylesuisse long live shellinabox  :stuck_out_tongue_winking_eye:


i'll delete the win32 project in a bit so we can start putting our actual code, note on the bottom right there is a button on the left bottom side of the website to download the zip folder if you don't download the program. otherwise just sync it to get any changes and just go to the source folder to make changes.


Here I changed Config and ConfigParser with Defs and DefsParser. ConfigParser is Python Standard Library module that I used later to build a central configuration point for the whole project.
With "Defs" I mean "Definitions", that is the STARTTLS enforcement policies to be configured in MTAs.


Huh? looks like this to me:
```D
Monitor *cs; // = null
if(cond)
{
    cs = something;
    ... // code without goto
    cs = null;
}

if(cs) // how can cs ever be non-null here?
```


Nit: driver.js fail_s_

Also, why does this happen exactly? Maybe there is a way to fix driver.js.


What's the purpose of this?


Replaced occurrences of hard-codec parameters with value read from the central configuration.


use `vconf`


Hi @arvidn, the file msg.hp still reference lazy_entry.hpp (instead of bdecode.hpp) and that's prevent a clean compilation.

Also the "result += <cflags>-Weverything ;" in Jamfile
and "logging" in kademlia sources in CMakeLists.txt
are creating compilation issues in darwing-5.1.0

Thanks!


Actually, I removed the dependency from the adapter package and not the adapter plus framework package, which we haven't been distributing. Of course, to avoid confusion, they should be the same.

I'm going to try to get away without having the adapter plus framework package for 3.0.


If some other thread sets `h.__monitor` at the same time, then it may not have a monitor when the function is called, but it may by the time `h.__monitor` is checked (whole reason for checking). But the code I identified will run both times. You need to move it inside the lock, or set a bool flag to run it if it was created. Well, it's not strictly necessary, running it twice doesn't hurt, it's just wasteful :)


say what type you got.

float. Got %s.' % type(train_time)


hi


This changed the of `*_changed?` on a patch release. This test passed on 4.2.0 but it fails on 4.2.1
```
diff --git a/activerecord/test/cases/dirty_test.rb b/activerecord/test/cases/dirty_test.rb
index 488813a..f5cae18 100644
--- a/activerecord/test/cases/dirty_test.rb
+++ b/activerecord/test/cases/dirty_test.rb
@@ -34,6 +34,11 @@ class DirtyTest < ActiveRecord::TestCase
     Person.create :first_name => 'foo'
   end

+  def test_changed_falsy
+    pirate = Pirate.new
+    assert_equal false, pirate.catchphrase_changed?
+  end
+
   def test_attribute_changes
     # New record - no changes.
     pirate = Pirate.new
```


## This is a markdown file


No, I pass the filter on to the query


3252d652a979dd31ed96945be4cdd07afa825632 hopefully brings source in line with what is currently live.


Nice to see you're getting started!
There's a couple of new features that have been added in the last week - d3 visualization of DAGs and return parameters from Chef recipes that can be used as global parameters (they are merged chef attributes of subsequent recipes).


Distributing them as dependencies would be a bit odd, I think, because (1) there is no dependency and (2) such a dependency would be the only difference between this package and the adapter package.

That's all part of why I dislike this package so much. Maybe we should just remove the source. We could always recreate it if we needed it.


yeah thanks :D that's about it


in the code above we ensured the cache is fully populated, so yes you are getting all tables available, but from the cache


.modman already is added and .git should already be ignored by git


why would you revert this?


Is `data` a Buffer or a String? If it's a Buffer I don't think you can concatenate it like this...


why revert this?


related https://github.com/rails/rails/issues/20110


having options that two kinds of selectors is helpful.  So,

parser.add_argument('-w', '--wavelength', .... ) then people can give either the -w or --wavelength on the Command Line.