@mikewest if you track something like client for navigations, open a ticket? Navigations is one of the areas that is poorly explored and also has interoperability issues. Perhaps now is the time that someone delves in...


Rules question: if the ability goes in the stack and before it resolves a creature is bounced, removed or whatever, should the ability fizzle on resolution? In other words, should the 5 creatures check be done both on the trigger and the resolution?


Comments about slow waves just before/after the operant actions, for SA14 and SA34


Here I think it's ok, but in future try to do it in separate steps.
1. Change (add fields + rake task for populating fields)
2. Cleanup (remove table)


Right, it should be in a comment on `\gresetdim` in `gregoriotex-spaces.tex`


OK. And in server is there a option to limits file?


ちゃんと File を close するか File.read つかうか `open(...) { |io| io.readlines.any? { |line| line.include?(uid) } }` かなあ


```
\return This function returns the local driver instance.
\retval NULL   Local driver instance is not found.
```


a line note 是什么啊？


translate name


Question, in which place you found this fix? Thanks !

Btw i am managing to make GCC 4.9 work, only bluetooth missing, soon will share :)


@lepture , why do use synchronous `XMLHttpRequest`?
[XMLHttpRequest Standard](https://xhr.spec.whatwg.org/#the-open()-method) says:
```
Synchronous XMLHttpRequest outside of workers is in the process of being removed
from the web platform as it has detrimental effects to the end user's experience.
(This is a long process that takes many years.) Developers must not pass false for the
async argument when the JavaScript global environment is a document environment.
User agents are strongly encouraged to warn about such usage in developer tools and
may experiment with throwing an InvalidAccessError exception when it occurs.
```


Oh, excellent, thank you!


Greedy貪婪演算法


從某一起點開始，不斷的改進，找尋周圍的最佳解，然後移至最佳解上，直到無法改進為止。


貪婪演算法會得到局部的最佳解，並非全域最佳解


Missing: copyrights


sys.dm_db_resource_stats should already be sorted by end_time, therefore we should already fetch the latest tuples .. I'm not sure if still it wouldn't be better to add a 'order by end_time' to the query... For the moment I left it out also for performance reason.


The function returns the processor instance if it is initialized.


Why this code is dead ?
Just to limit the twitter4j dependency ?

If this code works with the last version, i will create a new project to add other authentication mechanism.
Like OpenId Connect, google signin, ...


dissasociate user collection and wi collection. create wi for with same id as user id. we discussed this. images have wi items.


Assuming getScriptResource is called on every update on server...


merge


Assuming getResourceContent is called on every update on server...


this is a smart patch! can't understand why CM doesn't include this by default.


```
\retval kDualProcLast   The driver instance is not initialized.
```


Should the `sample_plugins` still be referenced here?


+1 column navigation should support infinite scroll


Rework function documentation like for the localProcInst!


Take a look here:
https://github.com/firebug/firebug.next/issues/385

Honza


Can you use the same indentation as the rest of the script?


なるほど修正します


```
\brief  Get the driver instance
```


I have added an issue for this: https://github.com/sulu-io/sulu/issues/1011


Wouldn't `CallHook` be better?


Change 'in' by at, because in this part we want to say that code will be deploy to your organization and not in your organization.


There seems to be some problems with the formatting in this section. The backticks does not display correctly.


@yuvalbar  why enforce alooma.io host in the code?


I'll be so happy when we make it to browser support and can drop client side.


I checked with @yuly-roberman and she reminded me that we need to send the "$result" object to the contributer so it won't be good to put in the kContextDataHelper .


:+1: Thanks, this let me add prime 9c48aaea90739a0465ac166ed2c7448dc4234200 and probably a lot of other things depend on this.


feedback queue


"українською" is correct, not "україньською"


The last 4 commits are not tested, but I have to leave home now.


@jasondegraw I may have found the error. I'll give you an update in a
little.  Don't bother looking into this until I talk to you again.

On Wed, Apr 15, 2015 at 9:04 AM, Craig Casey <c.casey.psu@gmail.com> wrote:

> @Jasondegraw I was busy trying to fix some other problems since this
> reply.  I am unable to get any error messages which sounds like it never
> even gets to rcontrib, since rcontrib would be producing the error file as
> a command line argument >>error.txt .  I attempted to put the full path in
> to get to the rcontrib program and that did not work either.  I will push
> what I have to the repo.  Let me know if you get a chance to look at this.
> I am going to continue trying to get this to work.
>
> Thanks,
> Craig
>
> On Tue, Apr 14, 2015 at 4:26 PM, Jason DeGraw <notifications@github.com>
> wrote:
>
>> That should be enough, but clearly not. Have you tried setting up a
>> standard error file to see if anything shows up there? Another idea would
>> be to temporarily put in the full path to rcontrib to see what that does.
>> On Apr 14, 2015 12:37 PM, "Craig Casey" <notifications@github.com> wrote:
>>
>> > @jasondegraw It is in the same place that oconv is in on my c drive.
>> This
>> > should find it the same way right?
>> >
>> > Thanks,
>> > Craig Casey
>> >
>> > Ph.D. Candidate
>> > Architectural Engineering
>> > The Pennsylvania State University
>> >
>> > IES 2015 Annual Conference Steering Committee Chair
>> >
>> > On Tue, Apr 14, 2015 at 2:14 PM, Jason DeGraw <notifications@github.com
>> >
>> > wrote:
>> >
>> > > That sounds like it isn't finding the program. You sure it is in the
>> > path?
>> > > On Apr 14, 2015 12:05 PM, "Craig Casey" <notifications@github.com>
>> > wrote:
>> > >
>> > > > @jasondegraw <https://github.com/jasondegraw> There seems to be an
>> > error
>> > > > here when running rcontrib. The error is "The system cannot find the
>> > path
>> > > > specified". Any ideas why this would occur from the STADICprocess
>> side?
>> > > >
>> > > > —
>> > > > Reply to this email directly or view it on GitHub
>> > > > <
>> > >
>> >
>> https://github.com/Architectural-Lighting-Simulation/STADIC/commit/16ba5e06d15eacc8c4bf19abba459f6a1e1cf49e#commitcomment-10720047
>> > > >
>> > > > .
>> > > >
>> > >
>> > > —
>> > > Reply to this email directly or view it on GitHub
>> > > <
>> >
>> https://github.com/Architectural-Lighting-Simulation/STADIC/commit/16ba5e06d15eacc8c4bf19abba459f6a1e1cf49e#commitcomment-10720212
>> > >
>> > > .
>> > >
>> >
>> > —
>> > Reply to this email directly or view it on GitHub
>> > <
>> https://github.com/Architectural-Lighting-Simulation/STADIC/commit/16ba5e06d15eacc8c4bf19abba459f6a1e1cf49e#commitcomment-10720683
>> >
>>
>> > .
>> >
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/Architectural-Lighting-Simulation/STADIC/commit/16ba5e06d15eacc8c4bf19abba459f6a1e1cf49e#commitcomment-10723437>
>> .
>>
>
>


Not sure we want to expose this URL here, double check with ops.


Is it `S` or `R` that starts the mapping?


I verified with @yuly-roberman and she said that after discussion with you (during the design) you both agreed that since the contribution is specific to getContextData this amount of generality is good.


As I said, I have verification code on picturit, that validates JPEG, PNG, gif, and svg I think. I'll port it to here. I'll also add jszip for the ability to upload sprites by extracting the assets locally, along with drag n drop, AJAX, and all that cool stuff.


s/times/time/


oi Rodrigo,

Esta abordagem com ArrayList resolve perfeitamente.

Mas tenta resolver utilizando HashMap.


s/review/reviewed/


- Do you really need that cast, because the array element is of the same type, right?
- Is paDualProcDrvInstance a pointer to an array, or an array of pointers? In case of array of pointers it should be renamed to **ap**DualProcDrvInstance


thanks. and just to confirm `select2-bootstrap.css` is being removed from 4.0?


I would avoid contractions such as "that's" or "it's" in written form; but I realise it is not unusual in the American language.


I submitted a PR. Just made a new branch to back out the schema changes,
because I was lazy and super tired and it was trivial.  Also fixed
"before_filter" to be "before_action" and whatnot, which is more
appropriate now-a-days (I'm old)

On Tue, Apr 14, 2015 at 3:02 PM, Andrew Hooker <notifications@github.com>
wrote:

> Without the schema changes (which don't appear to really be relevant
> changes), seems like this would be something that could be merged
> upstream...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/knewter/fat_free_crm/commit/7bcd54b7c4f3e9df62fe760655ccf4f11d14a6f8#commitcomment-10722336>
> .
>

--
*Josh Adams*
CTO | isotope|eleven <http://www.isotope11.com>
[cell] (205) 215-3957
[work] (877) 476-8671 x201

[mail]
3800 Colonnade Pkwy
Suite 140
Birmingham, AL 35242


"principal.m" is the file to run to obtain a list of artifacts and slow waves


Both, its an ‘Intervening if’. See http://blogs.magicjudges.org/rulestips/2011/09/the-intervening-if/


Any progress on this?


I've updated my previous comment so now you seem like a crazy person! (but yes, it is `R` and I was mistakenly writing `S`)


@thijsgillebaart I'm currently using a relative convergence measure for the PISO corrections based on the initial residual of the pressure.


Generally, do you need the remoteProcInst at all?


The last part is my fix, merged it together with your fix before I push the commit.


This is a simple and effective solution to the issue. IdPs that respond with an encrypted assertion without signing the Response element will break. No such IdP should be connected to SURFconext currently, as OpenConext/SURFconext does not publish an KeyDescriptor with purpose encryption in its SPSSODescriptor by default.


:+1:  :100: hot!


Not sure it gives me anything over Haskell vty-ui library. (BTW this README is old, checkout the hsversion branch). I would much rather not write Go. At all. :)


`#12345 Adjust constant [..]`

Do we need the `:` or is it just a source of confusion? We will see all kinds of variants including:

`#12345: Foo`
`#12345 : Foo`
`#12345 :Foo`


Nice...


What about deploy instructions?


for a simple test case, the speedup is 1.3 instead of using a fixed number of PISO corrections (5)


Is this not obvious from the commit message? Adding third-party code should always be done in an isolated commit anyway.


Ignore this change, needs to be reverted.  This was a quick test when diagnosing weird sql elapsed times that were showing 240 years.  :)


I would prefer that the instance_l variable is initialized in dualprocshm_create()


instead of inbox, this needs to take an argument, like 'feed' so we can do the same thing for any field we wish


Hey Remi. GitHub est le seul moyen que g trouve pour ecrire a un francais. et saches que je galere. orange ne marche pas. et je dois faire pleins de manip pour ecrire en francais， sinon ca donne ca. B啊东阿了Reminisce. je peux pas trop ecrire et je suis meme pas sure que tu le verras. saches juste que tu me manques bcp，que la chine et les chinois sont tres bizzards et que je n attends qu une chose， c rentrer. Sinon ca va =）
J essayerai de retourner sur github vendredi.
Je raconterai tout en details mardi.
Je t aime
Jeanne


CMSG_DB_QUERY_BULK: [Player: Gloriana (GUID Full: 0x08000400000000000000000000000005 Type: Player Entry: 0 Low: 5, Account: 2)] requested unsupported unknown hotfix type: 1821637041
CMSG_DB_QUERY_BULK: [Player: Gloriana (GUID Full: 0x08000400000000000000000000000005 Type: Player Entry: 0 Low: 5, Account: 2)] requested unsupported unknown hotfix type: 3326241424
CMSG_DB_QUERY_BULK: [Player: Gloriana (GUID Full: 0x08000400000000000000000000000005 Type: Player Entry: 0 Low: 5, Account: 2)] requested unsupported unknown hotfix type: 3386943305
CMSG_DB_QUERY_BULK: [Player: Gloriana (GUID Full: 0x08000400000000000000000000000005 Type: Player Entry: 0 Low: 5, Account: 2)] requested unsupported unknown hotfix type: 331613093
CMSG_DB_QUERY_BULK: [Player: Gloriana (GUID Full: 0x08000400000000000000000000000005 Type: Player Entry: 0 Low: 5, Account: 2)] requested unsupported unknown hotfix type: 3108775943
CMSG_DB_QUERY_BULK: [Player: Gloriana (GUID Full: 0x08000400000000000000000000000005 Type: Player Entry: 0 Low: 5, Account: 2)] requested unsupported unknown hotfix type: 599752208
CMSG_DB_QUERY_BULK: [Player: Gloriana (GUID Full: 0x08000400000000000000000000000005 Type: Player Entry: 0 Low: 5, Account: 2)] requested unsupported unknown hotfix type: 666345498


This possibly sounds like only http access is required when it is of course file access as well as http access.


+1

![image](https://cloud.githubusercontent.com/assets/1809908/7159559/0944b6a4-e351-11e4-8a3d-3603c0513cf9.png)


Clever solution!


I'm not sure if bridge is the right word since you don't use a bridge or at least you don't enable any hardware bridge, right?
Can you please check if there is any other possible name for that feature than "bridge" in order to avoid any confusions!


I would say It's faster for some as we do not have to fetch file information (if told so) and the resulting `CFileItemLists` are smaller. I think it's cleaner to pseudo pre-filter when possible. Is there a reason that your vfs implementation has this limitation?


wcześniej też INPUT nie byłe wewnątrz LABEL, a z BR to ok, niech strict - za daleko poszedłem


and it needs to be controlled so a set limit of ids are published at once


Should perhaps note something about how to add review fixes, i.e. as separate commits instead of squashing directly.


E.g. interface state (like in comment in line 207!)


@yjun2 done with my review. checkpoint complete.


@mseaton  I like it, thanks!  We will be definitely need to update the reference section of the reference or we will never remember that we've built this.


where does this go?Alias's, remote etc???whats the trigger


Should explain how to find the correct merge request given a Redmine issues.


add a mongodb array on wi object, with unique ids. whenever a field gets new data pushed to it, push the new id to this array (if it's not already there) and have the publish function on $in this array. this way we can publish a set limit of items and increase or decrese this as we need.


According to https://mambucom.jira.com/browse/MBU-7337 it should be possible to repost with an existing encoded key of the grouprole. Could you please raise a bug if this is not possible? Thanks!


- Missing *Dual* in tCommMemInst
- Add space between "}" and typename


Good catch, indeed!


j'ai ajouté un todo: Annotation sémantique --
---> veuillez voir ^^


- "CommMem" sounds cryptically, rename it to e.g. "CommonMem"


Checking the plugins directory is actually a very bad way to detect platforms, any other implementation could choose the same folder structure. Additionally we're going to change Vanilla's plugins folder to `mods` in the future for consistency between both implementations.


Could you please try to modify the group members here as well?


This would be moved to hardware/board/../.. include/dualprocshm-mem.h while integrating the pcie designs. Do you think, it could be integrated along with these changes?


@yjun2 done with my review. checkpoint complete. press on!


@homebysix Super cool. I'm a sucker for style guides and standards like this, so I'm in. Thanks for bringing this to my attention. Of course, I'll have to add a todo item to each of my projects to convert them all... A good chance to learn it of course.

Thanks!


Hey, this is a really nice project. I tired it in a demo project, but the ion-slide views could not scroll. Is this a bug or a feature?


I made a struct instead of a class because I didn't need methods or anything, just a single return value. A struct seemed like the simplest option.

embind allsows the use of `value_object` for structs, but trying to compile it gives me:

```
In file included from emscripten.cpp:8:
/Users/ahankins/Documents/code/devel/emsdk_portable/emscripten/1.29.0/system/include/emscripten/bind.h:772:23: error:
      implicit instantiation of undefined template 'emscripten::internal::GetterPolicy<mei::MeiDocument
      *mei::XMLImportResult::*>'
            auto g = &GP::template get<ClassType>;
                      ^
emscripten.cpp:66:6: note: in instantiation of function template specialization
      'emscripten::value_object<mei::XMLImportResult>::field<mei::MeiDocument *mei::XMLImportResult::*,
      emscripten::allow_raw_pointers>' requested here
    .field("meiDocument", &XMLImportResult::meiDocument, allow_raw_pointers())
     ^
/Users/ahankins/Documents/code/devel/emsdk_portable/emscripten/1.29.0/system/include/emscripten/bind.h:515:16: note:
      template is declared here
        struct GetterPolicy;
               ^
/Users/ahankins/Documents/code/devel/emsdk_portable/emscripten/1.29.0/system/include/emscripten/bind.h:773:23: error:
      implicit instantiation of undefined template 'emscripten::internal::SetterPolicy<emscripten::allow_raw_pointers>'
            auto s = &SP::template set<ClassType>;
                      ^
/Users/ahankins/Documents/code/devel/emsdk_portable/emscripten/1.29.0/system/include/emscripten/bind.h:554:16: note:
      template is declared here
        struct SetterPolicy;
               ^
/Users/ahankins/Documents/code/devel/emsdk_portable/emscripten/1.29.0/system/include/emscripten/bind.h:778:33: error:
      implicit instantiation of undefined template 'emscripten::internal::GetterPolicy<mei::MeiDocument
      *mei::XMLImportResult::*>'
                TypeID<typename GP::ReturnType>::get(),
                                ^
/Users/ahankins/Documents/code/devel/emsdk_portable/emscripten/1.29.0/system/include/emscripten/bind.h:515:16: note:
      template is declared here
        struct GetterPolicy;
               ^
3 errors generated.
```


Is Pcp/Host known in the dualprocshm? In tDualProcInstance there is first and second!?


The binding is:

    value_object<XMLImportResult>("XMLImportResult")
    .field("meiDocument", &XMLImportResult::meiDocument, allow_raw_pointers())


Note sure this is really intended as user facing documentation (also interesting to see this is more extensive/explicit than when we added ``Bio.Entrez`` to Biopython, it looks like we might need to register our Entrez tool name string with the NCBI)


This value was in here because the tests expect this exact value but you might've changed your local `config.js` file


Replace with https://www.chromestatus.com/feature/4529989986811904 so it links to individual page.


You rock! Thanks

Matthew

please excuse typos - sent from my phone.
On Apr 13, 2015 9:29 PM, "Shanbin Wang" <notifications@github.com> wrote:

> I deployed version 1.1.2, which should be available in update center in a
> few hours.
>
> http://repo.jenkins-ci.org/releases/org/jenkins-ci/plugins/scoverage/
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/jenkinsci/scoverage-plugin/commit/42eb91cba9b389bbb96aa8041019ec95c80a3333#commitcomment-10704880>
> .
>


Done.


For relevant Vim help files, check out `:help Operator-pending` and `:help :map-operator`.

Summary: the bizarre way to define your own operators which accept a motion is to make a a binding which first sets `opfunc` and then invokes the `g@` operator. The `g@` operator waits for a motion and then calls `opfunc` with information about the motion.


Is it possible to add a default value to prevent BC breaks ?
Something like `$direction = 'from'` should do the job. What do you think about that ? @mmeloni
\cc @lphuberdeau


Can we end these questions in `?:`


Was that the patch making it ready for merging?


Sectence -> Sentence


Huh! I didn't know toUpperCase was locale-dependent...! Well, at least Build and Push will work in Turkish deployments now :D


`(node.tagName == 'A') && (node.className == "invalid-subtitle-link")`.  Is this part of the check necessary?  If multiple sources triggered onSubtitleLinkClicked, you could handle it by passing in `($event, action)` and have action be different for the different sources.

Not a big deal though, I think this way is fine as well.


I was prefering TEE_ERROR_GENERIC. But then I would alos return GENERIC in propset_get() function above.


It seems it is better to use objects for sessions like $dumdum.


...maybe I'll write a blog post about this.


Correct to autocapitalise. You say Beacon here :)


Beacon here too. Correct plz :)


Why is enabled that specific value?
-> I would prefer having a magic field and using some interface state where 0 is disabled and not 0 is enabled


Yep, thanks for fixing my fix. I think I have to warn you guys that I was not aware of this and there's a high probability that other cards I've done have the same issue, unless melvin spotted them when submitted. I'll try to check them later, although I'm not too sure how to do that efficiently in this case. I've recently downloaded the current comprehensive rulebook to try and get up to date with the rules, but without playing live actively it's hard to memorize things.


+add test8 on robell_git for fetch test


This change is pretty much abandoned. Check feature/multi_auth instead.


This commit produces following errors on master side  of synapticon/sw_sncn_motorcontrol_ethercat_kit/app_demo_master_cyclic_velocity
Failed to write SDO!"
Received 4 bytes do not fit info SDO data memory (1 bytes)!


Why is this a convenience methods? It looks like the standard implementation.


> What about another branch specifically for f-droid without the g+ api?

I'm not really sure if this would be a burden to maintain. Here's how did the author of QuickLyric solved this problem (by using product flavors): https://github.com/geecko86/QuickLyric/blob/master/QuickLyric/build.gradle (also have a look at the projects src folder).

> And what if I enable google drive api in future, does the same apply to it?

As long as it requires non-free library (like PlayServices) - yes, it wont get into the F-Droid repo.


Please add the mandatory note to the other mandatory parameters as well, thanks!


essa solução embora mais simples não garante o resultado da busca em TEMPO REAL.

isto é, se um elfo tiver seu status alterado após ser alistado no exército, essa busca vai estar desatualizada!


The idea behind time.time() was that if there was no end for the transition then it should extend beyond the report end (timewise).


właśnie robię poprawki.
"Myślę, że te trzy case-y da się prosto zwinąć do jednego kodu z trzema case-ami na początku." - tu to nie wiem jak :( to są moje początki w programowaniu


`my $moff = shift;` would seem a bit clearer here.


This isn't intended behaviour. It should be returning 20 commits as that is how many are created in the setup script: https://github.com/domharrington/node-gitlog/blob/master/test/create-repo.sh#L13


Thanks for the feedback. Yes, I am very aware of the bad way to detect the platform Foundation is running on, and was looking for an alternative way. I keep an eye on the Sponge repos, so this garbage can be removed as soon as possible.


I wasn't sure that it should be user facing either, but since it *is* an eutils based tool, and should conform to NCBI's requirements, I figured it was best to have this be made extremely explicit so that users would be well informed (and I could cover myself in case NCBI is unhappy).


You are right. I have initialized it in lines 148 and 165. This comment is more of a heads up as to where this variable is used.


Okay.
Returning an error code can be handled by the caller to avoid a crash(on AP side). To completely avoid system crash, additional changes in handling the PDO buffer allocation and stack initialization would be required.


Implementing the changes you've mentioned - though this is a really handy one. I've put it into place for this (which is now in a 'shared/task' partial).


Aren't parts of this GPL? Are we not using any of the GPL parts?


fair enough


I changed this for a reason, there was a bug caused by using time.time() here.. I really need to start writing better commit messages, haha.


Done - not sure why I went off piste for this. All fixed and RESTFUL now.


Thanks for adding these annotations, @ihnorton – very helpful.


I figure my next best option to mentioning timing is to do an initial count on "efetch" results, and if it's >1000 sleep the job until 5pm/weekends...but that's pretty anti-user.


Thanks, done.


I know the BnP code is already littered with this kind of repetition all over the place, but let's try not to perpetuate the style. We can move the ```if(isGzipCompressed)``` block outside of the ```if(saveKeys)``` block and avoid duplication. We should also probably refer to constants for the suffixes (".data", ".index", ".gz").


Please do it in an extra commit for the current available hardware designs (xilinx-z702/mn-dual-shmem-gpio is the only one right?) - if it can be changed along with this commit I can squash it before integration!


Yep, good catch.  Removed as an aid to debugging.


It was getting too crowded, and the reddit link is on the Community page.


This whole if can be simplified to ```conf.setBoolean("save.keys", saveKeys);``` for better readability.


`commiter` is incorrectly spelt here. It is `committer` with 2 t's as documented here. http://git-scm.com/docs/git-log#_commit_limiting


Oh, that explains it! I've been half-wondering about that `g@`/opfunc thing for a few years. I'd read that blog post.


Same comment as above.


Ahh right yes, now I remember. It was causing problems with the regression tests. So not quite a bug in that sense. The regression test output would always differ since the current time changes.


:clap:


Same


Is this a really big faux pas (I've a feeling it is)? It works, but happy to fix if this is the wrong way to go about this.


forgot to edit the comment


Small typo in this and the line below -> Oerdraft


double randomDirection = Math.random()*2*Math.PI;
particles.add(new Particle(clickPoint, color, d * Math.cos(randomDirection), d * Math.sin(randomDirection));


"... PDO buffer allocation and stack initialization ...": Which functions in stack in particular?


Might be sensible - but ideally any logic like this would be done at job scheduling otherwise the long sleep just hogs a cluster node pointlessly. One of the reasons I didn't start work on any Entrez/Galaxy tools myself was this kind of worry.

Am I right in thinking Entrez Direct does its own load limiting? I would hope so, which makes this the NCBI's own problem to deal with ;)


Yes. We dont need it part as part of the request. I will fix it.


Right, missed that. Added note.


Tu pourrais utiliser les mails, comme tout le monde ^^


We're at least using SPQR which is GPL, didn't check further.


yes, hibernate didn't like the fact that the BaseModel is not defined in hbm.xml, and I've read into doc that the object hierarchies for two different ways of definition must be separated.. I will try once more since I took it for granted before final testing :-)


It would be helpful to have something like this checked in permanently so we can debug failures (especially if they end up being sporadic). What if you only do the print if the number of events is incorrect? The test will fail anyway, so I don't see harm in there being a printout out.

In fact, you could create the string and pass it to the assertEqual as the message for the failure.


I'm wondering if this and the line below should rather be set globally in the static constructor / class initializer.


Not sure, really, as I don't quite understand what "AllowAll()" is for in the first place.
It's a videodb vfs (with url like "videodb://tvshows/titles/529/3/11689.mkv").

Thinking about it, it's probably wrong, indeed, in this case.
So, if you don't think it might be an issue for other vfs'es, just forget about it and sorry for the spam.


There seems to be a lot of duplication between this and ```HadoopStoreWriterPerBucketTest```? Are they copy-pasted?

Please consider merging the two tests or extending a common class, if doable. Otherwise, let's at least add a warning in both class headers that any changes made to one test should likely be copied into the other as well.


dein GIF ist unter jhoveValidations/src/uk/gov/nationalachives/droid/GUI/DROID16.gif
statt
jhoveValidations/src/uk/gov/nationalachives/droid/GUI/Icons/DROID16.gif


It is quite complicated because all the packages of the SuiteSparse have different licenses and even the modules of e.g. CHOLMOD have different licenses. I'll try to compile a list of what we are using and the licenses.


@Kovah Could you maybe clarify why this RewriteBase is here? It's causing some issues when the installation is not in the root directory, but everything works perfect without it.


Ideally this can be retrieved via the product API.


Here's the Debian license summary, in case it helps: https://launchpad.net/ubuntu/vivid/+source/suitesparse/+copyright


Про changelog забыли


I initially thought this is also where the rpc operations picks which node to send it. Later I realized its not needed, but forgot to clean it up. I will fix this in next iteration.


Intro to ReadMe File


Yes, you are right in that aspect.
Here the loop was added to avoid crash/error on AP(host) side if PCP has not yet written the values of the buffer offsets, due to buffer allocation/initialization delays. This happens usually for PDO initialization, when the wait time on host side is not always enough for PCP to allocate and write the buffer offsets, after host has posted the init command.

Currently, for PCIe based designs, we are using a loop with timeout here. Do you have any suggestions?


We should try to avoid whitespace changes (whether caused by auto-formatters or otherwise), as it makes diffs and the git history noisy.


Probably remove them from git history too


I will


Wondering the same


Hmm, yeah. Makes me think there are a couple of things which should go into workflow/job scheduling:

- tool needs internet access flag (for easy sequestering of jobs onto a head node for data download)
- multiple copies of this tool cannot run at once

As for load limiting; `Bio.Entrez` does some already, it behaves to the limit of "no more than 3 requests per second" (though, given how long it takes to schedule and execute Galaxy jobs, that isn't exactly a problem).


Should be an abstract method, to be overriden by Impl classes


Yes, you are right. Similar kind of discussion followed between me and Gaurav :)


Hello,
I'm still getting this error:

```
Preparing for installation...
  └── Creating project installation path...
Installing Backend...
  ├── Installation path is c:\work\git\memyne\alpha\backend...
Downloading and extracting node-seed...
  ├── Creating local configuration file config/local.json...
  ├── Installing NPM modules...
Error: spawn npm ENOENT
Status @ 5s |   Installation Progress: [▒▒--------]  21 %   ┊   Installing: Back
end   ┊   Step: Installing NPM modules...   ┊   Step Time: 1s   ┊
```

I'm on Win7 and I've node 0.12.2 and npm 2.7.4.
Thanks


Okay.


Should the default be none? I think it makes sense to require an explicit config change in order to change behavior...


Okay. Thanks.


Why have you decided to do move InputMaskDefaultMask out of  InputMask


@mihairaulea better to not commit system dependent paths


Finally, to set the limit to 2 processors, this was added.


If there is a problem then the "throw" statement will return the error code and the error description.


@raprasad please note that @sekmiller has indicated that "category" is a legacy field at this point that will probably be dropped at some point. Right now it's always null and I heard from @landreev that when you sort by it you get random sorting.


А не могли бы вы подсказать как вы дебажите? У меня получается только если установить брэйкпойнт и запустить через "run->debug 'server name'." Но так компоненты не подебажишь. Не могли бы вы описать краткую инструкцию а то уже давно ковыряю и все не как. Заранее благодарен.


Are we log rotating these?


Hm


env is not defined here anymore. Should either be env=os.environ or removed.


command -> cmd


@rhb-anme done with my review. goodness, you have come so very far in just a few weeks. the effort is commendable and I thank you for it.


@grossmj  Are you sure you want  drop iourc_path? The tests was broken after that and the JSON schema still include it. We can probably drop it but I want to be sure.


this could waste the chance for other scrub jobs to be scheduled. the larger `(info.history.last_deep_scrub_stamp + cct->_conf->osd_deep_scrub_interval) - (info.history.last_scrub_stamp + cct->_conf->osd_scrub_min_interval)` is , the more chances are wasted.


- Do you refer to the timeout in pdou.c line 990 ?
- Maybe you can implement a loop in pdoucal_allocateMem() in pdoucalmem-noosdual.c? Or have I missed something?


As @bradly mentioned, we are not but should be. If anyone wants to open a PR for that ... :+1: :smile:


The top description mentions that a push.node is specified somewhere but I don't see that anywhere. Not sure if the discrepancy lies in the description or in the code, but anyway. I think the check should happen on the push.cluster (i.e.: any node in the VIP). Alternatively, we could make the check happen on all node (to ensure consistency), but I don't think we should go there just yet, as it is a bit of an orthogonal change...


:beers:

![](http://media.tumblr.com/6fa6206356358b58c881293d086e8507/tumblr_inline_mfu42hcrvt1qdb4er.gif)


OK


Sure, looks good.


Okay


Good ideas. On our setup the cluster nodes should still have internet access, but putting that as a flag would be nicely flexible.

If Galaxy knows that multiple copies of this tool cannot be run at once, that would go a long way to obeying the Entrez rules. As you note, Biopython does the "no more than 3 requests per second" but will not be aware of other jobs running on the cluster/node also calling Entrez.

(I've not checked yet, but would expect at least the EFetch tool might end up making multiple requests for batch downloading)


There are hooks that have special treatment, it would be unfortunate to have those use a different call mechanism.
You might be able to use specializations:
```cpp
template <int hookType, typename... args>
bool Call(Args &&... args)
{
  // Generic code
}

template <cPluginManager::HOOK_SPECIAL_HANDLING>
bool Call(/* args */)
{
  // Special handling
}

// Calling the hook:
cPluginManager::Get().Call<cPluginManager::HOOK_PLAYER_JOINED>(*m_Player);
```


Sure


Okay.
Yes, in cases where the local processor needs to access the shared memory regions of the remote processor, especially header.
In our case, the memmap module uses this to get the remote processors shared memory base address.


Merge après modifications ( on tourne dans le sens trigo + makefile absurde )


Okay.


I completely forgot about passing strings to the `assertEqual`! Good call!


Finally it appears that we can continue using BaseModel with Installation!


> On Apr 15, 2015, at 8:08 AM, Laurent Pugin <notifications@github.com> wrote:
>
> This is an offset in the octave calculation that remains from Wolfgang code. Originally this was used to reduce the amount of memory used for storing the octave in an unsigned char.
This is nostalgic! I remember doing little tricks like that!

Ich

> I moved everything to one place and made it a CONST (see afd8b90), which it indeed should have been in the first place...
>
> —
> Reply to this email directly or view it on GitHub.
>


The tests don't expect this exact value anymore.


Ненене, все не так. Ты борешся с симптомами вместо болезни.

Тут хватило бы следующего.
Поменять
MCLocation.current.distanceTo(location);
на
MCLocation.getBestFusionLocation(context).distanceTo(location);

А current вообще к херам снести из MCLocation


Wow, people can sometimes be creative when writing python code, can't they? :)


Wow, it looks like a lot of progress here!


So will this be in test 14 ??


I meant to type "case insensitive" not "all lowercase" lol


elif?


we can'd drop "category" - maybe we call it something else (for example, tagging of a data file), but many users heavily rely on it to group/categorized their files and sort or filter by them to download only files from one category.


don't forget to cap at 15 :)


Noen grunn til at disse blir resolvet her og ikke i `server.js`? Virker litt rart å sende inn objekter istederfor paths til noe som forventer seg options.


add


case 'pppoe': case 'dhcp': case 'eap':
  a potem wewnątrz takiego bloku wszędzie zamiast konkretnych odwołań po prostu zmienna a której opiera się switch case.


No, that's not needed.
It is an array of pointers, would be changed to ap.


Correct URL: https://groups.google.com/forum/#!topic/django-users/agvMBs8DKEM


Oh, by the way, adminer is on GitHub now, so our repository should be a fork of that one, with a branch for our modifications (so we can still pull and send them PRs from our main branch): https://github.com/vrana/adminer/


a to lipa była z punktu widzenia ergonomii


Nice :)


A+++++ COMMIT


There are a bunch of problems with a bunch of things related to this. Let's just say you'll be getting less of the test set than I thought you would...


So forgot about the testing code here.

Travis is going to succeed, because it'll install your module before running unit tests.

So the best way I've found to work around this is running.

```python
sys.path.insert(0, os.path.abspath('..'))
```

Which places your root package in the absolute path so you can then use absolute imports and that's all you would need to change.


It's .html, not .md: http://bazel.io/docs/windows.html.


chaitu:shaju:afeya:subash


@tarelli yes, that is so; it slipped, sorry. anyhow, it should be stored in an xml or in a config file.


Lots of code duplication here... there is about 3 lines of code that differs between the two branches of the if... I think the code can be made a lot more readable by extracting the common parts outside of the if.


I'd prefer `s/Dynamic/dynamic/g`. And how do I build without avx2? When I use `./configure --disable-native-tests`, I get a build that doesn't even have SSE2. Do I have to use `-f Makefile.legacy` for a build with dynamic formats and acceptable performance?


Comprobad que todo funciona


@Mrcl1450 now i'm fix all :D . you can fork and sync it !


Yep, efetch splits requests out into batches of 200 records at once. https://github.com/galaxyproject/tools-iuc/commit/8da6943b0c91102db5e8da2c800666127ac27d35#diff-c90933e75fcd7db002e89eeee4588a0bR37


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


You should just include your editor specific files in a [global .gitignore](https://help.github.com/articles/ignoring-files/#create-a-global-gitignore)


Set spectator-enabled to false, because this hasn't been tested yet!


[^1] is already taken on another slide, you need to update the number with the next available one


I found this solution by [Brian Kent on SO](http://stackoverflow.com/questions/21323170/how-to-resolve-scala-2-10-x-flatten-issue).


I'm starting to think that this feature is patched on " no vanilla " servers.
Think how many people got scammed by the 10 Euros client xD .


@brianpfink does this include the fix for the inaccurate limit fill behavior you noticed?


Why not prefix increment? Its simplier in disassembly, right?


Addressed in the child commit. Saw that as soon as I pulled up the diff. Thanks for catching that!


fixed in  8b814db


nodejs_fixes (lefticus) - x86_64-MacOS-10.9-clang: Tests Failed

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


Then maybe something generated HTMl from MD? If you look at https://github.com/google/bazel/blob/master/site/FAQ.md#what-about-windows and click on "using MinGW/MSYS" (which links to https://github.com/google/bazel/blob/master/site/docs/windows.html) you'll get a 404. My commit fixes this.


@mcrosas "category" was moved to a different spot in the code I mean.


On Wed, Apr 15, 2015 at 06:39:55AM -0700, Kristoffer Grönlund wrote:
> Ahh right yes, now I remember. It was causing problems with the regression tests. So not quite a bug in that sense. The regression test output would always differ since the current time changes.

OK. Anyway, for that there's a variable...
options.regression_tests. It'd be better to:

if options.regression_tests:
etc


I would be in support of this blog post existing and would certainly be willing to help. Also explaining the 4ish conditions (blockwise, visual, linewise, and characterwise) handling would be good. Per @gabebw https://twitter.com/gabebw/status/587748704318316544


Then people would not be complaining about it on spigot forums.


The usual way to do this is `USER_AS_A_CASE.enabled(domain)`


@jjangsangy I am not quite sure what you mean.


ack


Maybe you could add this to other releases (Daisy...)


I'm fine w/that if other people are, but it seems on the long side IMO, even for an integration test.


Trello cards here:

- https://trello.com/c/PYYNVhkC/2631-mutually-exclusive-jobs
- https://trello.com/c/6UuGxdh4/2632-add-flag-to-tools-denoting-internet-access-is-required


This is a bit of a weird place to put this since the user will still see the user case properties on the UI but if they try and build the app it will fail. I think for now the best approach will be to disallow adding the user case properties on the UI if the toggle isn't enabled AND the it isn't enabled on the domain.


@madmaxoft Unfortuanatly not.You can't specialise function templates, only class templates. You can use an inner class, but that just gets messy.


Oh, of course. Thanks! Sorry, I disable that for local planemo testing as the packages aren't in the repo. Nice catch.


Nice idea :)


Open to discussion on appropriate toggling behaviour.


I would rewrite this slightly:

```python
user = UserFactory.create()
factories.TextCommentFactory.create_batch(2, user=user)

self.assertCountEqual([user], models.Groups.objects.users())
```


Remove duble blank space that is before of and word in the second line.


修改了地图界面的中心点设置，添加一键导航按钮


Thanks. It helped a lot. We are using many of the GPL parts and it is not really feasible to restrict the use to the LGPL parts. If we want the GPL parts out of Base, we should probably consider doing it as part of 0.4.


DNN-6865 when a user exists in more than one portal and one portal gets updated to use emailAsUsername, the internal username is updated at next user's login and is no longer allowed to login with username in other portals.


If this is intended for tests only, then perhaps consider making the constructor protected and put the corresponding test class(es) in the same package name.

It would also be good to make the comment proper javadoc (i.e.: start it with ```/**```) so that the warning about the intended scope of its use (i.e.: test only) is visible through javadoc.


these refs were on removed slides, will update accordingly if we reintroduce them


Change was by were when you talk about two classes


Yes, our documentation is served using Jekyll, which converts .md files to .html.  Thus, you generally cannot follow links from our GitHub pages.


Can you merge this change to master so I can benefit from it?


Swell nothing special really. On my migration I was just using another method to add it.

Your migration is fine, but you should probably add an index as well, so instead of:

add_column :sequences, :recipe_id, :integer

do

add_column :sequences, :recipe_id, :integer, index: true


There are two blank spaces before you word. Remove one blank space


ack


Need a person, who can solve it.


But the type casting from an unsigned type to a smaller unsigned type is implicit; adding an explicit cast doesn't change anything. So I wonder, what is the motivation for this?


@RushOnline : Thanks for this but this is the wrong place for bugfixes.

`libcryptoplus.debian` is merely a repository for the Debian package. The upstream is now part of [`freelan-all`](https://github.com/freelan-developers/freelan-all).

You can see the same line (with the warning fixed) [here](https://github.com/freelan-developers/freelan-all/blob/master/libs/cryptoplus/include/cryptoplus/bio/bio_chain.hpp#L98).


Sure, no problem. Thx anyway. Will test them when I find time ...


'Label' is misspelled. Please make the proper correction.


Be sure to use BLE module with BGLib_U1A1P_38400_noflow_wake16 firmware (https://github.com/jrowberg/bglib/tree/master/BLEFirmware/BGLib_U1A1P_38400_noflow_wake16) with this software, as wakeup hangs if it's not connected.


okay, fine!


0) I still do not understand why you had to change "mat" files. Please explain.

1)In ellintersection_ia

The following block is incorrect as absolute precision should be taken from ellipsoidal object as self.getAbsTol().

modgen.common.checkvar(inpEllArr, ...
    'det(x.getShapeMat()) > elltool.conf.Properties.getAbsTol()', ...
    'errorTag', 'wrongInput:shapeMat', ...
    'errorMessage', 'Shape matrix of ellipsoid shouldn'' be equal to zero.');

Also, instead of 'det(x.getShapeMat()) > elltool.conf.Properties.getAbsTol()' string you can use a function handle like this:

@(x)(det(x.getShapeMat())>absTol)

Please do not write self.absTol in the function handle as it may keep the whole ellipsoidal object as a context.

Finally, it is not correct to use "det" function here as we already have a special function for checking that a matrix is not degenerate

gras.la.ismatnotdeg. In gras.la package there are very many other useful functions.

2) In intersection_ia the following change

if doesIntersectionContain(ell, poly)
    [x0 E] = getInnerEllipsoidParams(poly);
    outEll = ell.create(x0, E);
elseif ~intersect(ell,poly)

is incorrect as x0 and E are incorrect name.

For getInnerEllipsoid it was ok to use bad names because it was copy-pasted from old mpt toolbox. But in our code
we cannot use "x0" and "E".

Also, what was a point of removing "ell.create" from getInnerEllipsoid?

3) I do not quite understand why you made elltool.core.GenEllipsoid.reDecomposition static.

It should remain non-static method, just rename it into getCopyWithReComposition. You should not be confusied with the way
this mehtod is called as in matlab getCopy(ellObj) has the same meaning as ellObj.getCopy().

Please revert this change, just rename rename it into getCopyWithReComposition.

4) You continue to ignore my request to keep all lines under 74 symbols, please fix that.
Also in

            for i = 1 : nFirstElems
                ellFirstChangedArr(i) = ...
                    elltool.core.GenEllipsoid.reDecomposition(ellFirstChangedArr(i));
            end
            for i = 1 : nSecElems
                ellSecChangedArr(i) = ...
                    elltool.core.GenEllipsoid.reDecomposition(ellSecChangedArr(i));
            end

Please rename "i" into "iElem" and use import

5) In GenEllipsoid line 124 in

        function shapeMat=get.shapeMat(self)
            shapeMat=self.eigvMat*self.diagMat*transpose(self.eigvMat);
            shapeMat = (shapeMat + transpose(shapeMat)) / 2;

we should use (shapeMat + transpose(shapeMat))*0.5; instead of /2

6) Please avoid simplifying error tags by renaming "wrongDiagonal", "wrongCenter" etc into "wrongInput".

Instead just replace "wrongDiagonal" with "wrongInput:wrongDiagonal" which will solve the problem with the tests.
Otherwise we will make our error tag less informative.

7) In GenEllipsoid on line 479 please replace
                    if ~ismatposdef(ellObj.diagMat(isNotInfIndVec,...
                        isNotInfIndVec),absTol,1)
with

                    if ~ismatposdef(ellObj.diagMat(isNotInfIndVec,...
                        isNotInfIndVec),absTol,true)

We should not use numeric values intead of logical values


This performs no sanity testing on the output from the NCBI. They unfortunately have a track record of returning HTTP 200 OK and no data (personal experience), or a plain text error message rather than real data, e.g. examples from @kblin on Twitter recently https://twitter.com/kaiblin/status/587540788617306113

    .@NCBI: Case in point, you're returning 220 OK and "Resource temporarily unavailable" in body right now. Please return status 503 instead.

and https://twitter.com/kaiblin/status/588262945303486464

    Also @NCBI, nice work in inserting "Ressource temporarily not available" errors in the middle of a GenBank file via Entrez

I've seen similar problems myself - including one error message with a v/w spelling mistake (presumably from the legacy system which Entrez was linking to) which I tweeted some time back.


In the interest of getting this finally merged I'll assume that this is indeed to silence a "truncation warning" from the compiler. But please motivate your changes.


yes it does, that logic is encapsulated in price_worse_than_limit_price.


TeamCity xBIM :: XbimExchange :: Build [Build 219](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1397&buildTypeId=XBIM_XbimExchange_Build) is now running


It is supposed to be that. It's just weird in my workspace.


Okay, please change it to **ap**DualProcDrvInstance**_l**
(**_l** ... since it is local variable!)


how are you aiman


You will need to make sure this is properly deleted. That is calling
   XMLImportResult *result = mei::documentFromFile
will then require the caller to do
   delete results


No problem, thanks for keeping at it.


TeamCity xBIM :: XbimExchange :: Build [Build 219](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1397&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:01:26


The DISCARD mount option should not be enabled in Android 4.3
and later for partitions that the OS trims automatically.
The presence of this option slows down the device.


On 04/15/2015 04:06 AM, sijchen wrote:
>
> thank you very much for the info. Actually we can wait a few more time
> before getting this in, because there is still one TODO to complete
> the correct setting.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sijchen/compare-codecs/commit/57f1687ade2fa4a9299ad3bbc1fae93cd3065ed6#commitcomment-10727439>.
>
(it's inconvenient to have only communication via this PR. Can you send
an email to harald@alvestrand.no or hta@google.com?)

I've also tried different RC values; it seems that "-rc 2" gives better
compression / PSNR than all other settings, but the result is totally
non-responsive to the "bandwidth" parameter.

This is passing strange.

Harald


![Uploading 1apple-touch-icon-precomposed.png . . .]()


The issue see to be to have a class (a pointer to it) within your structure. I would just make it a class


I see, so this basically is https://github.com/github/pages-gem/issues/69, thanks for the clarification. I'll close the PR.


Would it be better if I changed the error messages to `"Iterables must not have more than two items"`/`"Iterables must not have less than two items"`?


change 'into an organization' by 'to an organization'


@dstung please help me test these new fields. Thanks


nodejs_fixes (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


s/Wether/Whether/


Can you take me off this list please? I'm not able to help on this project
anymore.

Thanks,
Michael Zaro

On Wed, Apr 15, 2015 at 7:58 AM, Matjaž Horvat <notifications@github.com>
wrote:

> Correct URL:
> https://groups.google.com/forum/#!topic/django-users/agvMBs8DKEM
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mathjazz/pontoon/commit/4207aee5f914f57b51572684a64cd143cf703913#commitcomment-10736423>
> .
>

--

Michael Zaro
Partner, Verisage <http://verisage.us/en/>

CEO, Dojo Dev Camp <http://dojodevcamp.com>

[image: Twitter] <http://www.twitter.com/michael_zaro> [image: LinkedIn]
<http://www.linkedin.com/in/michaelzaro/>


This is necessary since construct now always expects an rvalue. And the lvalue dummydest cannot be assigned to this rvalue. The only other way I see of fixing this is to.  In c++11 to having the dest argument be a "dest_t" instread of the current "dest_t &&" (in c++ 98 we have dest as "const dest_t &") but with having dest be "dest_t" it is somewhat hard to ensure that all the pipe segments move constructs. Perhaps we should delete the copy constructor for pipe segments in c++11 mode?


Probably we don't want to add it to the unflushable count. This is because we don't differentiate a pinned object to be dirty or not. If we add the num_pinned to the unflushable, we need also to decrease the dirty pinned object from the total dirty objects, which is not convenient to do for now.


good you ?


Yeah, exactly.  Sorry for the inconvenience!


> And how do I build without avx2?

That's a separate issue, but we should document it briefly in doc/INSTALL. I use to do variations of
```
./configure CC="gcc -mavx" --disable-native-tests
```


The important part is `protocol=2` as that's what pipe uses. The `__getattr__` hack is not needed for protocol 0 and 1 :confused:


Added ma self


We could use errno.EOPNOTSUPP instead of 95, to use the errno symbols.


This is currently possible, isn't?
https://wiki.galaxyproject.org/Admin/Config/Jobs?highlight=%28job_conf%29#Limiting_Job_Resource_Usage


Жанр лучше подзапросом выбирать даже,
то есть:
SELECT m.id, m.title ..., g.name
FROM movies m, genres g
WHERE (lower(m.title) LIKE lower(?) OR ...) AND m.release_date LIKE ? AND ? IN {
SELECT gm.id
FROM movies m
LEFT JOIN genres_movies gm ON gm.movie_id = m.id}
Последний (?) заменяется на id жанра, который передается из view


to avoid the call to Math.abs(), just revert the order of operands in your subtraction, as NOW >= LAST_REORG_DATE.


Typo: ""UNKOWN" state"


nodejs_fixes (lefticus) - i386-Windows-7-VisualStudio-12: Build Failed

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


PS I think the best would be to return it by value (and not by pointer address)


b


LGTM


> Why are you removing the breadcrumb-like functionality?

Actually I only looked at first line `content_for :pagetitle`, it means that it was some way to set the title of page but we weren't using variable `pagetitle` anymore, we just use `title` now. Hence I removed the complete block and added [this](https://github.com/sonalkr132/GlitterGallery/commit/4c85ad413b573bd28eae8111cfbac38c064ff716#diff-ff6b3642f5484a5f08445873acd65224R13). I didn't mean to remove any image link.
> I think that we should have links to the user, project and the image

Link to project is under `project_toolbar` (which is already included). Adding `project_header` should instead of what I have done should fix issue of link to user, but before I push with that change..
> Isn't it good to link to the original image?

What exactly is the history of an image? I have raised concerns in my PR [#287](https://github.com/glittergallery/GlitterGallery/pull/289). Is updating the image1.jpg with image2.jpg allowed? (notice change in filename) If it is shouldn't we remove image1.jpg from our current state of repo? Can a user update a single image with upload of multiple images? Tho broken, thats what our master seem to have right now check this [project_con.rb](https://github.com/glittergallery/GlitterGallery/blob/master/app/controllers/projects_controller.rb#L272)


Looks good, but could you also add pynotify to the extras dependencies in setup.py (under "other").


Here is a quick test:
<pre>[2015-04-15 14:22:45] [bootloader] DEBUG: Set GRUB_THEME="/boot/grub/themes/Manjaro-Default/theme.txt" in /etc/default/grub
[2015-04-15 14:22:45] [bootloader] DEBUG: Set GRUB_CMDLINE_LINUX_DEFAULT="resume=UUID=6a0cb357-64f7-4b7a-b5ae-531ba815e7fa quiet splash" in /etc/default/grub
[2015-04-15 14:22:45] [bootloader] DEBUG: Set GRUB_DISTRIBUTOR="Manjaro" in /etc/default/grub
[2015-04-15 14:22:45] [bootloader] DEBUG: /etc/default/grub configuration completed successfully.
[2015-04-15 14:22:45] [bootloader] INFO: Installing GRUB(2) BIOS boot loader in /dev/sda
[2015-04-15 14:22:45] [bootloader] DEBUG: grub-install command: grub-install --directory=/usr/lib/grub/i386-pc --target=i386-pc --boot-directory=/boot --recheck
[2015-04-15 14:22:48] [chroot] DEBUG: Installing for i386-pc platform.
grub-install: error: cannot find a device for /boot/grub (is /dev mounted?).
[2015-04-15 14:22:48] [bootloader] INFO: Installing Grub2 locales.
[2015-04-15 14:22:49] [bootloader] DEBUG: 50mounted-tests file patched successfully
[2015-04-15 14:22:49] [chroot] DEBUG: /usr/bin/grub-probe: error: cannot find a device for / (is /dev mounted?).
[2015-04-15 14:22:49] [process] ERROR: Couldn't install boot loader: [Errno 2] No such file or directory: '/install/boot/grub/grub.cfg'
[2015-04-15 14:22:49] [process] DEBUG: Exception: name 's_dir' is not defined. Trying to continue.
umount: /install: target is busy
        (In some cases useful info about processes that
         use the device is found by lsof(8) or fuser(1).)
[2015-04-15 14:22:50] [process] ERROR: Command '['umount', '/install']' returned non-zero exit status 32
</pre>


Hello @atavism,

The suggestion about the 8 was just an example to see if it changed anything. If that's so we could think on a long term solution. I like @myleshorton's idea but I'd apply it to the main config routine since that's the one that seems to be producing the blocking.

Did that suggestion about the buffered channel changed the outcome at all?


I think --preserver-temp-files should be mentioned in the --save-prefs option, a bit further down.


@bgruening my reading of that is that those are global limits, "the number of concurrent jobs users can run", not "the number of concurrent jobs per specific tool that users can run"


@KonstantinKo: Gibt es einen schöneren bzw. effektiveren Weg die Reihenfolge der Kategorien in der Sidebar der Suchergebnisseite denen auf der Startseite anzugleichen?


There was some reason I did it at that stage, and now I can't for the life of me remember why.


There's some weird stuff with gulp-jscs not catching this from the command line. At least on my machine


essa solução embora mais simples não garante o resultado da busca em TEMPO REAL.

isto é, se um elfo tiver seu status alterado após ser alistado no exército, essa busca vai estar desatualizada!


yes


Does this need to be included in the upstreaming patch?


will do


Ed, what's the best way to share this Error Messages partial across different classes? This seems to work, though it feels very wrong to include the logic here.


Muito boa a solução!


Missing?


This is a typo from trying to get depth handling to work properly. It should have been `-1` only.

The depth handling code is somewhat ugly, but it means that the initial version can avoid touching some things where regressions would be very bad and such refactoring could be decoupled from the initial implementation of the stable API at the expense of a few extra CPU cycles.


Awesome,


yes you're right :baby_chick:


Correct. The thought of sanity checking NCBI's data for valid fasta/xml/asn1/text responses seemed like a nightmare. If you have suggestions here, I'd be happy to entertain them.

Their API has a lot of really nasty corner cases and absolutely terrible failure modes. I've seen similar errors on blast (`-remote`) results where they give up partway through. I've also seen WebEnvs take time to propagate, you cannot just `epost` and `efetch` immediately afterwards, there's a sometimes large delay between the history becoming available for querying even though you've already been given a presumably valid ID.


We have to check if mounting is still needed. I got an error cos of that with current git code.


Does anyone dislike the idea of supporting both keywords?


I'm fine with that.


(To me the fact that their API is miserable says more "we need a OSS replacement" rather than "we need to put extra work into dealing with a service that doesn't even have a bug tracker")


oi Rodrigo,

tenta resolver este exercício sem ter vários HashMaps para cada status.


Abel Garcia


Yes! Every bit used to count! Thanks for pointing it out


You mean because the pref gets saved? --preserve-temp-files is a runtime pref, doesn't get saved in preferences


Kannst du Boob Design auch im CSV readden?


Hago commit


ok


Yes getting the right set up can be a little tricky. Sorry about that.

Good you figured it out.

-Andi


Delete this. Messed up making a new branch.


@bgruening turns out you're correct, my apologies. @jmchilton mentioned a solution for that using a special destination.


No problem


Ok, don't know.  `registered_user_concurrent_jobs, unregistered_user_concurrent_jobs, concurrent_jobs` I assumed `concurrent_jobs` is global.  @natefoo ping! :)


Will set Null as default then :)


\o/


Actually this works (for some time it did not)
```
./configure CFLAGS="-g -O2 -mno-avx"
```
I will add this to doc/INSTALL.


legal


Aight, would like to get these questions answered before merging.


This complicated conditional is a workaround for my desire not to touch `dmu_objset_find_dp` too heavily. Opinions on whether this is a good choice would be great.


isn't `empty` enough?


Why the different strategies here? `MapMaker` v. `CacheBuilder`


Oh, and output to COM1 is added as well.


nodejs_fixes (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Tests Failed

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


I know this is nitpicking but you make this `final`


Is F2FS g2g?


def 和if、while、for语句一样，在末尾必须有个冒号（英文的）


Still not right.  `-print` should go before the limit

`"%s . \\( %s \\) -prune -o -type f \\( %s \\) %s -print %s"` is correct


13行和后面的几行缩进没有对齐


@phollegien: il semble qu'il y ait des problèmes d'encodage avec ton ajout.


lgtm


Unless you pass --save-prefs, I expect? In any case, --save-prefs just has a list of options that modify the preferences, even if just temporarily, so this option shoujld be listed there, I think.


@floriank not sure if this fits in with the original intention of your choice component. Works ok though.


Thank you for the reminder. I had actually created a local branch with that behavior when I made this test. Though as you say, as far as I know it's not used. As a result I didn't merge it.
I'll attempt to find time today to get it on here.


如果input函数中输入的a和b是整数的话，那么b/a结果就不是我们所期望的，应改成b*1.0/a


Yes with special destination you can set the 'need internet flag'.


FWIW, we need a proper data source for NCBI. This would be a clean solution to this entire problem. Hans-Rudolf and me was discussion this recently and we will try this at least for EBI, SwissProt ...


@jmchilton said

> I believe one should be able to define a destination just for these tools in job_conf.xml and set the concurrent_jobs on the destination to 1 in the job conf limits.

which means **everyone who installs this** will have to add the following to their `jobs_conf.xml`

```xml
<destination id="entrez" runner="local">
</destination>
<limit type="concurrent_jobs" id="entrez">1</limit>
<tools>
  <tool id="ncbi.eutils.efetch" destination="entrez" />
  <tool id="ncbi.eutils.esearch" destination="entrez" />
  <tool id="ncbi.eutils.epost" destination="entrez" />
  <tool id="ncbi.eutils.elink" destination="entrez" />
  <tool id="ncbi.eutils.einfo" destination="entrez" />
</tools>
```


Yes, full sanity testing would be a nightmare. e.g. For some formats an empty return value is probably OK.

The Entrez API is workable, it just needs to use HTTP Error Codes when things fail. Sadly we can't add an open source replacement given this is the only official way to access the NCBI's online resources.

Personally I'm open to adding some basic sanity testing to Biopython's efetch (peeping at the returned handle for a black list of known NCBI error messages), but I'm not the primary maintainer of ``Bio.Entrez``.


改成这样
```
#print语句可以用逗号将要输出的内容连接起来，而不是点号
print '太阳仰角C=', C
```


WIP not ready !


`CacheBuilder` makes it easier to atomically: check if something is in the cache, and if not generate a value from what we have currently in scope and put it in. I've used `MapMaker` where we didn't need that, because it's simpler.


Neue Häuser comitted!


另外，以后提交的时候，不要点“sign off”那个，在填写”提交描述“那儿写点中文，比如对代码做了什么：删除、修改、新增等等


Thanks!


Why do we need to care about iterables that are not collections (i.e. that have no len)?


Now has a dedicated repository: https://github.com/jhlq/Equations.jl
Will make another specifically for non-commutative geometry


Additional notes: this doesn't seem to be necessary if we are setting `siteurl` properly; it should handle the redirect to https for us. Open to explanation why this might not be true. :)

The problem this solves is that to cURL and Firefox, this would create an infinite 301 loop to the https version of the site, no matter your entry point to the main page.


@noplay I thought this isn't used. Let's keep it if it is still needed.


Thanks. This really needs to to above line 14 though. The paths are tried in order, top to bottom.


Hi Bill, thank you very much for comments.   In Tree.js I removed additional definition for _setLabelAttr() and added code to existing. I added automated test for "rename"  to ProgrammaticTree.html  and removed Tree_node_rename.html, that is not needed now. I commited and pushed the code. Please take a look. Thanks


Indeed, I was planning to put collect some snippets and put them on github. We have collected over time some nice examples and created some `dynamic_job_destinations` that might be worth sharing.


Here I get this: ERROR: cannot open `/tmp/phprvSnO9' (No such file or directory). This is also saved in the database and displayed in the HTML link element, type attribute.
When I use the variable $type from above, from the line 129, that contains the same information, it works fine :-)
:-O


I think it's this line that meant that `MapMaker` wasn't enough.


奥，谢谢老师，我立马改


This could be nicely reused in WinForms as well, but that would need some common helper, or shared source. What do you think? Struct or class? Class is easier to use, struct has better footprint but using it is a bit harder (some methods need to be static with `ref Helper me`). I'd prefer class - struct would be overkill.


@bgruening yes, definitely.

@peterjc yeah, no good solution there. Implementing some sanity checking upstream in biopython would be great. I'll open a bug there for tracking


one is tracking the consensus ledger, the other the state in the database, so yes, we need both


looks good


From the Sublime Text 3 docs:

> Shows a quick panel, to select an item in a list. on_done will be called once, with the index of the selected item. If the quick panel was cancelled, on_done will be called with an argument of -1.


you might want to do this for the date comparison check... not sure how effective what you have there will be.

```
if ( strtotime( $details->registered ) > strtotime( '2015-04-15 12:00:00' ) ) {
 add_filter( 'singl_custom_background_args', 'ee_single_custom_background_args' );
}
```


File deleted


Thanks for the reminder, @majshehab. Added #64 for the 15 cap.


It would have been cleaner to separate the software updates for the affected issues, wouldn't it?


Incluir nuevo nombre Abel


The only tricky thing here (although, the answer might be "we don't care") is that we generally send Stackdriver customers to support.stackdriver.com links and GCM customers to something.google.com ones.  But in this off-chance that's very unlikely to be hit, I think that just sending to a support.stackdriver.com link is reasonable for now


Thanks, I still have to write a script to do that.


@bgruening  You have? Please, please do!

I'm...not happy with job_conf.xml, I've been considering for a while writing a d3.js interface to configuring it, it's just so painful and there are so many different configuration options that can be accomplished there.


Yes, we are either going to be delegating to another repository to handle the Bootstrap theme or pulling it into the core.


Hi Michael,

I think you need to do that for yourself in GitHub notification settings.


I see 2 different cases:

- the admin want to limit the number of concurrent jobs on a destination to N per user
- the admin want to limit the number of concurrent jobs on a destination to N in total

I'm not sure which one is implemented by `concurrent_jobs` (according to the Wiki, seems to be the first one, but John seems to imply the second one instead).


You may drop the `pypi-signed` role and its keys.


good point, thanks


@xiam No, it didn't really resolve the issue (config updates still block after a few changes). I agree we should apply it to the main config routine! Do you want to look into that if you get a chance? I know this is already affecting the beta without this PR, and I'm still working on these FireTweet updates.


that makes sense. thanks!

Zaro

On Wed, Apr 15, 2015 at 8:48 AM, Matjaž Horvat <notifications@github.com>
wrote:

> Hi Michael,
>
> I think you need to do that for yourself in GitHub notification settings.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mathjazz/pontoon/commit/4207aee5f914f57b51572684a64cd143cf703913#commitcomment-10737493>
> .
>

--

Michael Zaro
Partner, Verisage <http://verisage.us/en/>

CEO, Dojo Dev Camp <http://dojodevcamp.com>

[image: Twitter] <http://www.twitter.com/michael_zaro> [image: LinkedIn]
<http://www.linkedin.com/in/michaelzaro/>


This cannot be cached, as css rules may change at any given time. Aka if md-select is hidden (directly or not), line height will evaluate to 0. Thus always always shrinking text. All I can see is '...'.


As you can see [here](https://github.com/arduino/Arduino/blob/master/arduino-core/src/processing/app/PreferencesData.java#L146-L148), keys starting with `runtime` are explicitly omitted when saving prefs, so --safe-prefs has no effect when enabling --preserve-temp-files


Consider new color scheme!


They're not valid any more.


Uh oh. Ruby time.


Hi Magnus, thanks for jumping in! I'm using the latest version from master (edenhill/librdkafka@13a1bfe), and still getting this error when the message body is NULL:

```
%3|1429108960.163|FAIL|rdkafka#producer-0| localhost:9092/0: Receive failed: Disconnected
%3|1429108960.163|ERROR|rdkafka#producer-0| localhost:9092/0: Receive failed: Disconnected
%3|1429108960.163|FAIL|rdkafka#producer-0| localhost:9092/0: Connection closed
%3|1429108960.163|ERROR|rdkafka#producer-0| localhost:9092/0: Connection closed
%3|1429108961.167|FAIL|rdkafka#producer-0| localhost:9092/0: Receive failed: Disconnected
%3|1429108961.167|ERROR|rdkafka#producer-0| localhost:9092/0: Receive failed: Disconnected
%4|1429108961.167|METADATA|rdkafka#producer-0| localhost:9092/0: Metadata request failed: Local: Broker transport failure
```

I'll try digging into it further, but if you have any ideas on how to best debug it, please do say.


Done.


hello, chat now?


Shouldn't this come from ApplicationController instead of been defined here? How do we know it caches in controllers?


Won't this change now wrap AuthenticationException in a ConnectionException?  Will that cause any problems?  Don't suspect it will but thought I'd point it out.


Hmm, yeah the right thing for me to do would have been to make sure this was the same actual commit in the other PR, and not just cherry-picked commit with the same diff. I needed this commit in the PR still for tests to pass. Now that that's merged my only options are (1) do nothing and leave this here and just let reviewers they don't need to review this particular commit, and (2) to rebase, which would orphan all comments on commits so far. I'm going to just go with 1 as an imperfect but simplest solution. Sorry for the inconvenience!


You  don't need to delegate to a `pypi-signed` role.  Having just the `targets` role handle all of Seattle's targets should be enough for now.


I'm just not sure about doing 'as SilexApp' - seems unnecessary


So, I'll play around with this, because I'm also not very good at maintaining tests.

But ultimately here's the skeleton

```
root:
    setup.py
    ...
root/tests
    __init__.py
    fixtures.py
    ...
root/libextract
    __init__.py
    coretools.py
    ...
```

So currently you have 2 different packages, which are seperate together. Your `./test` directory and `./libextract` are both separate modules.

So running `python setup.py install`, installs two packages for you. One package called `libextract` and one package called `tests`. That is why your testing code like this succeeds.

```
from tests.fixtures import element
```

https://github.com/datalib/libextract/commit/4f84f5ea947c1c4ec5801062c193d715ea72029a#diff-17010c64b196adee658e95f058483aa9R1


Since we are removing defunct from other places, shouldn't we also remove it at line 122 as well?


Ah, indeed, so effectively it isn't really a proper preference I guess. I wonder if this needs some documentation in itself? Probably not...


Perfect


Got it.


Now that we multiple rows per community this update should run in single transaction:

```ruby
FeatureFlagModel.transaction do
   flags.each { |feature, enabled|
   #...
```


Okay, as soon as someone else agrees with our view we can do that of course.


Suggestion: End this line with a ':' instead of '.'.

"This requires some additional druid configuration:"


good！加油！


Sure, let me take a stab at it.


Currently, as Mson is built up, we only put a JsonArray of Mson objects into our object. The Json looks much cleaner when not all msons are packed into each other and are instead put into this List.

The purpose of this iterator feature is, that we can set all parts of the more mson easely. Sure, we could just make this a normal function, but the code looked really clean that way in the set methods.


There's a typo on this line.  This should have a comma and not a semi-colon.

rcommentValidation = new RegExp("(?:FieldName|FieldInternalName)=\"" + columnName + "\"", "i"),


Oh god this is going to be really miserable to debug.


This is a fix to #539


Add-on Documentation/Performance & Monitoring/Blackfire.md

Does this mean I need to be logged in before clicking the SSO button or will I just be asked to sign-in?


Could you please provide a installation manual or the jar to import this theme as settings?


Forgot to get rid of it in my hurry probably :)


I'm getting this error via `gulp` and `gulp watch`  on  my linux machine.

BTW hershey is missing in the package.json


I made it runtime exactly to avoid documentation. It's used only when using the CLI and leaves no traces


@runrevian I started testing this commit out and found a few things. My test involved compiling a widget using the extensionBuilder using the "Test" button and then testing "popup widget" in LCS. The following LCS script is triggered from a message posted in the OnClick() event of another widget:

```
put "hi" & cr & "-" & cr & "what" & cr & "why" into theA["menuText"]
popup widget "com.livecode.extensions.screensteps.themedMenu" at pAtPoint with theA
set the clipboardData to the result
```

1. After 'popup widget' 'the result' contains binary data. I am using 'close popup returning "hi"' to dismiss the popup.

2. Escape key doesn't dismiss the popup like it does for a system popup.

3. Clicking in a location outside of the popup doesn't dismiss the popup. Rather than popup disappears quickly and then reappears.


how to make this jump cause i really dont know i think i try most of options


TeamCity xBIM :: XbimExchange :: Build [Build 220](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1398&buildTypeId=XBIM_XbimExchange_Build) is now running


More clear class name needed


Use StringHelper::basename


TeamCity xBIM :: XbimExchange :: Build [Build 220](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1398&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:01:04


Suggestion:

"Allow migrating segments from one place to another. Currently this is only used as part of the MoveTask to move unused segments to a different S3 bucket or prefix in order to reduce storage costs of unused data (e.g. move to glacier or cheaper storage)"


@PascalPrecht this commit breaks the current functionality (run button) for me. The previous commit seems to work just fine and also looks better as it places the panes horizontally next to each other. I wanted to do a quick prototype of the save/restore feature while I'm waiting for my flight.

For now I'll just use the previous commit as a starting point and rebase it once master is in a functional state again.


[R/sumstat_ihh.R:42:40:](https://github.com/paulstaab/coala/blob/20c91ea11a5666cd3c93109cb2278f1c790efc95/R/sumstat_ihh.R#L42) *style:* **Place a space before left parenthesis, except in a function call.**
```r
        rehh::scan_hh(haplohh)[snps , -(1:3), drop=FALSE]
                                       ^
```


Да уж, current вечно под ногами путается. Делай тогда сам :)


Would it make sense to state this documentation is primarily about how to integrate cloudControl and Blackfire and refer to the respective sections of the Docs on your page for configuration options etc.?


it gives error while downloading lp5.1 blisspop rom


@nsoranzo according to the wiki, you can limit with the following `type` values:

- registered_user_concurrent_jobs
- unregistered_user_concurrent_jobs
- concurrent_jobs

we, of course, choose the last


But theres no milk in the game


The changes above make the subsequent "used extensively" comment a lie, since `lapack.jl` is still using the old style.

Notably absent from this section is any explanation of how to get the data back out of a `Ref` object; it looks like the cleanest way is currently `width[]` and `range[]`.


"Why would you want the first one in the list to be the root element?"
The Json looks nicer if we don't have one completly empty Mson as a root element. :) less hassel to detect mal formed Json when testing. :)

Taking chat color was a feature we thought might be nice. This way, you wouldn't have to call the methods for it and you can build the Mson just with putting the objects you wish it to contain. This was before also applied to formats, but now we should probably get rid of it.


http://forge.prestashop.com/browse/PSCSX-5317 Not all of those tags work on og:type product. What about using object type conditionals and use smarty template includes and make specific templates for various object data types?


I looked more closely at `dd_create_skeleton`, and I see that it uses `g_mkdir_with_parents` internally. This means that it will succeed even if the directory exists. `dd_reset_ownership` is called before the `rename` operation to drop the temporary `".new"` suffix. The directory name contains the PID, and the kernel may prevent parallel execution of the coredump handler for a process with the same PID because the PID is still in use until the coredump handler closes standard input. But these are quite many assumptions.

I think you should really use plain `mkdir` and fail if the directory already exists.  It's also not clear to me why you need to lock the directory while it still has the `.new` suffix.


I am well aware that this is not standart java coding style. But as the Mson class is currently built up, directly etting the field is not what you/we wanted with the set behavior, which is why it is called new. We never call this directly outside of Mson. More explanation further down.


What is the purpose of this addition? If `cell` is a `UITableViewCell`, then it's a subclass of `UIView`, which defines the `superview` method. This seems unnecessary.


It *is* weird! This is the root of this entire month-long quest I've been on. Jsonobject internals (and couchdbkit before it) routinely strip the timezone off incoming datetimes. In fact, it even *matches datetimes strings with a timezone* (if they're not in the schema) with a regular expression and aggressively converts them to datetimes—again stripping the timezone and microseconds. Here I'm bringing this dark little nugget of filth out of its hiding place and exposing it for the world to see: here I explicity strip the timezone off without converting it, keeping the exact same behavior as before. Except now, I don't strip timezones, and I'm using [a `DateTimeProperty` that respects microseconds](https://github.com/dimagi/dimagi-utils/pull/266/files#diff-c3c6a581dde068d3a55dd27d0ee8cf1aR31) and fails hard on timezones rather than converting them (and [doesn't aggressively match them](https://github.com/dimagi/dimagi-utils/pull/266/files#diff-c3c6a581dde068d3a55dd27d0ee8cf1aR71)). So now we do this dirty work up front, and then pass it to a version of jsonobject with cleaner, stricter datetime semantics.

The next commit I'm making on this code will check the `TIMEZONE_DATA_MIGRATION_COMPLETE` toggle and, if it's set to true, convert to UTC instead of stripping—which is the crux of the behavior change and has to be (automatically) coordinated, per domain, with the data migration.

Hope that helps!


@Underground78: Opinion on this? It will print the same output as before. If exception ocurred trace is printed on screen and pause is invoked.


@nsoranzo my reading of the wiki is that `concurrent_jobs` is N total on a destination, as the `<limit/>`s also provides an option for `registered_user_concurrent_jobs` which would be the first case, N per user on a destination


My first project in github


lrn2rebase pl0x :P


Will upload this asap.But why the hell we have 2 version numbers?


@exstar I think this is a very good "interim" system for detecting the Sponge platform.


Not sure if this was just overlooked or fixed later, but this will not run because in rows 5 and 12 there is no `phaseMask` only `phaseId`. Once changed it runs.


This code is pretty mysterious to me.   The `mycompare` function calls `a < b`, but `isless` is not defined for `Ref{Cdouble}`.   Is `cfunction` performing some undocumented magic here?


@XhmikosR @Underground78: Should I use "High precision timer" as suggested by ever_green on trac? I think it is not important since this menu refer only to timer.


好像有人用“渐变色系”，我觉得色系这词应该争议不大，而“渐变”比“连续”个人觉得好理解一些


https://github.com/lmfit/lmfit-py/blob/master/lmfit/asteval.py

See lmfit Interpreter class as a more extensible interpreter and parser for the expressions to be passed through this function.


The docs for `gs_console_begin_status_line()` state
```
You must invoke gs_console_end_status_line() to return the console
to normal mode.  In particular, concurrent use of this function and
the stream returned by gs_console_get_stdout() results in undefined
behavior.
```
I think I was seeing that undefined behavior leading into my first attempt.  Sounds like I need to temporarily end the status line before printing GPG results.  I'll try playing by `GSConsole`'s rules.


The references to "type-tag" are pretty obscure.


Hmm, I wonder why I never saw that. I'll try to fix, but probably not today. Thanks for reporting the problem!


sample


@erasche `registered_user_concurrent_jobs` is not per destination!


The user doesn't have (and can't) configure anything. That said, two links could be useful: https://blackfire.io/doc/web-page and https://blackfire.io/doc/first-profile


Ah, sorry for the confusion!

On Wed, Apr 15, 2015 at 10:03 AM, Philip Durbin <notifications@github.com>
wrote:

> @mcrosas
> <https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_mcrosas&d=BQMCaQ&c=WO-RGvefibhHBZq3fL85hQ&r=n9HCCtgqDPssu5vpqjbO3q4h2g6vMeTOp0Ez7NsdVFM&m=nhpwOR_dRtRupCNGDqqsbosT2DQ8TL2o92gtkZF3ppc&s=mqmcuB8j0PH_Pu5bzkQpxDjYjjOAsbk-3-FVvg7dn-s&e=>
> "category" was moved to a different spot in the code I mean.
>
> --
> Reply to this email directly or view it on GitHub
> <https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_IQSS_dataverse_commit_380863a7b15a6fb45823c62018870149c92107f6-23commitcomment-2D10736516&d=BQMCaQ&c=WO-RGvefibhHBZq3fL85hQ&r=n9HCCtgqDPssu5vpqjbO3q4h2g6vMeTOp0Ez7NsdVFM&m=nhpwOR_dRtRupCNGDqqsbosT2DQ8TL2o92gtkZF3ppc&s=uIR7TVl0IK24QbjB-HR0gkozGNkfL9AMaWGZe3pcoKY&e=>
> .
>


Hi!
Is this tile working properly for you?

Pause?
Play the 2nd time?

I have some issues, so asking before I start playing with it


Improvement suggestion:

"DataSegmentArchiver: It's just a wrapper around Mover which comes with a pre-configured target bucket/path. Thus it's not necessary to specifiy those parameters at runtime as part of the ArchiveTask."


Note that the value could be a `PortRange` if this is a `PortAttributeSensorAndConfigKey` for which only the sensor part has type `Integer`


You know, it was probably just timing of when I made these fixes—I made `iso_string_to_datetime` part way through so I wouldn't have to do this. I'll make a commit on top of this to replace all `DateTimeProperty().wrap` with `iso_string_to_datetime`. (I'm making a concerted effort not to rebase this branch anymore, as that will only cause confusion now that you're reviewing.)


Looks good! Can you make a few minor changes, though? Can you fix the spelling of "minutes" on line 664 and document the fact that `latest` will take precedence over `minutesAgo` if both are specified?


Granskad!


Nah, seems redundant I think since it's obvious it's the timer.


@nsoranzo sure it is!

/me goes to read wiki ...

Oh. Well that's bloody annoying.

`(limit attribute id is a ) destination id. Destination on which to apply limit (for concurrent_jobs type only).`


It's working fine now.


Suggestion: Replace "a good idea" by "recommended".


Hi Dan:

I unfortunately do not have the time to devote to this. Feel free to take what you like. Thanks for doing the work we built upon!

Jade


Looks ok


I wrote some methods in security Controller which generates report based on the selection.
I also included one folder named as gwt in resources which contains all the gwt stuff related to reporting.


http://www.dituhui.com/help/color 我觉得关于三个色系，这里的命名似乎值得参考，我之后再结合全文考虑一下


Why loop twice, once through `getEntityType().getConfigKeys()` checking for `PortRange` then here through `config().getBag().getAllConfig()` looking for `*.port` key names? Should we have both checks in both loops, or just one loop with both checks?


What's `test`?


not sure what this is for here...


These aren't the same.

    after      before  rightValue   leftValue ResultValue
    -----      ------  ----------   --------- -----------
     True       False          -1          -1           1
     True       False           1          -1          -1
     True       False          -1           1          -1
     True       False           1           1           1

     True        True          -1          -1          -1
     True        True          -1          -1           0
    False       False           0          -1          -1
    False       False           0          -1           0
    False       False           0          -1           1
     True        True           1          -1           0
     True        True           1          -1           1
     True        True          -1           0          -1
    False       False          -1           0           0
     True        True          -1           0           1
    False       False           0           0          -1
    False       False           0           0           0
    False       False           0           0           1
     True        True           1           0          -1
    False       False           1           0           0
     True        True           1           0           1
     True        True          -1           1           0
     True        True          -1           1           1
    False       False           0           1          -1
    False       False           0           1           0
    False       False           0           1           1
     True        True           1           1          -1
     True        True           1           1           0


First argument to Collection is the Model and 2nd are instances.


Sure. Submit a PR.


Row 8 there is no `phaseMask` but `phaseId`.


Glöm inte att ta bort onödiga console.log innan du pushar.
Granskad!


Puedes probar el funcionamiento en esta etapa sin problemas


Actually both are possible: if you are already logged you will be authorized instantly, if not you will be asked to log in or register and then you will be authorized (no other action should be needed).


FWIW this broke the Windows version of Etherpad...


Hi, yes tried it now. Works without issues. Pressed Play and Pause ~7 times and it works everytime


this is perfect, so dont worry what they are telling u to do. this is correct!


Even dubbel check via product owner. Dan mergen we hem.


Yeah, had a good convo with @ctsims about this, and this was by far the simplest satisfactory solution


[tests/testthat/test-sumstat-nSL.R:13:15:](https://github.com/paulstaab/coala/blob/e6769d5f872df843fc64693f181f7a1b1aa98cce/tests/testthat/test-sumstat-nSL.R#L13) *style:* **Variable and function names should be all lowercase.**
```r
  stat_nsl <- sumstat_nSL()
              ^~~~~~~~~~~
```
[tests/testthat/test-sumstat-nSL.R:20:15:](https://github.com/paulstaab/coala/blob/e6769d5f872df843fc64693f181f7a1b1aa98cce/tests/testthat/test-sumstat-nSL.R#L20) *style:* **Variable and function names should be all lowercase.**
```r
  stat_nsl <- sumstat_nSL(position = 0.5)
              ^~~~~~~~~~~
```


even it's set, it has no effect and can't controlled from outside easy.


PHPCS: Line exceeds 120 characters; contains 125 characters


Line 64 `phaseMask` - `phaseId`


We should update the project page asap: http://gitshots.com - Let's make this viral :P


its not going to work, you havent imported getdirection


Enligt den kodstandarden vi följer ska det vara mellanslag mellan if och parantesen. Annars bra.
Granskad!


Yes -this is logic that should be in the controller


Curtis, this is technically wrong: autogenerated files should never be written to the source directory.  The buildbot is supposed to be setting the source directory as read-only, so that during build, the writing of files to source dirs is prohibited...


Okay, so let's try to test it!


Granskad!


In test/test_files/example_problems/CREC-2013-01-23-pt1-PgH251-3.xml

Line 20 and following of the corrected, inserted, text. I am seeing that those names are being tagged as speaking when they are just being appointed to a board. I also see that the middle name followed by a period is being identified as the end of the name, whereas it should be after the last name that the tag closes.


LInes 5 and 2865 `phaseMask` should be `phaseId`


nodejs_fixes (lefticus) - Win64-Windows-7-VisualStudio-12: Build Failed

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


@erasche @jmchilton We still have a problem about `concurrent_jobs`, either the Wiki is correct (limit is per user, and your use case is not solved) or John is correct (limit is on the total).


just `return pm.podByFullName[...]` or am I missing something?


Sorry, I somehow thought these ones were seperate! Will take more notice next time ;)


end parenthesis is not properly aligned with the others


No. The name is misleading, and I fail to understand what it actually does. The comments tell me JQuery's `replaceWith` might be useful, though. http://api.jquery.com/replacewith/


Not as in the current build up (comment further up explains it), no. Using the List for it's purpose instead of packaging all Msons into each other, this is the way to apply it to all parts.
The code would not be less if the were packed into each other, because the checks would be the same and the code would just "loop" itself


same


@runrevian Another issue I'm seeing has to do with widgets being updated after calling 'popup widget' from their LCS script. Try this:

1. Add the Cross Platform Button widget to a stack.
2. In the mouseDown LCS script of the widget call 'popup widget'.
3. Click on the button. You will see the issue I described before where clicking outside of the popup causes the popup to disappear and then reappear. This only seems to happen with widgets. The same mouseDown script in an old school button works fine.
4. If you make a selection from the popup widget the widget disappears but the Cross Platform Button widget instance state isn't updated. It still appears in the Down state. It's like OnMouseCancel/Up/whatever isn't sent to the widget.


maybe too many indentation levels


The @error object should not be tied to a specific resource. It should be independent. and assigned in the controller in the case of an error.

The simplest way however is just to use simple_form that will handle all this stuff for you directly in the form.


I wasn't done yet with that part either. Don't worry, I'll get to it.


no need to have a return just have

```
false
```


:D


Also, it needs to be rebased on the last master before submitting.


It could accept a collection for sure :+1:


is this working. u are missing and end


Tänk på att merga filen riktigt. Några grejer till behörighetsstyrningen försvann här, men jag lägger upp dem igen.


@fabpot : shall I create an I issue for this?


Can we RLock no?


you probably want a custom model method here. something like

```
unless task.matches_team_for_user?(user, team)
  # code
end
```

not the question mark at the end of the method - this is standard for when return a boolean value from a model method. much easier to read


Line #103 the double :: was intended for the output, but I agree that it would look better without, I wanted a easy way to tell if it was seconds.


Accepted


Chrooting should happen after **self.auto_timesetting()**, please double-check. alsa-mixer throws now lot of errors.


Granskad


by default, the map returns only one arg, the value.
so you have to do this ugly thing to tell it to return both value and ok.


JSON_SUBSCRIBER should be JSON_PUBLISHER


Agreed! Thanks @hinerm! One refinement would be to use `ImageMetadata#getBitsPerPixel()` in at least some cases, to further narrow the default channel max in cases where you have e.g. 12-bit data stored as 16-bit.


No god mode in RESTConnector


lots of chaining here. better to abstract all that logic out into a single model method e.g. in team.rb

```
def uncompleted_tasks
  tasks.includes(:team).where(complete: false)
end
```


oh. sorry.


Erm, I'm actually stuck again.

I've rebuilt the entire toolchain with latest HJ's binutils with LTO patches and enabled --enable-plugins from the GCC, but I still don't have 'ld-plugin' binary.

Any ideas?


@Stefano80 : Was it intentional in your test (http://tests.stockfishchess.org/tests/view/552d67580ebc596367ee9915) that only "new" and not "base" is running with nodestime=600?


Not sure what to do here. I think the best would be to just remove the exception altogether and drop the expires field in the cookie in such a case.


I have an issue related to this fix and appendOffset.top in bootstrap3:

https://github.com/eternicode/bootstrap-datepicker/issues/1364


OK, so to summarize, this doesn't fix the problem, it just moves to out of
couchdbkit so that we have more control over it and can do a gradual
migration of the data using the toggle. That makes perfect sense and clears
up a few things. Thanks.


this is a little bit sad that those classes aren't uniq for javascript. It's nice to differentiate js classes from css classes `.js-roles` for example.


This still seems more scary than it probably is. Do we even need to show this if it is recoverable?


thank you! took this one off my shoulders!


地图配色会显著影响你运用数据所诉说的“故事”，认识到这点十分重要。本节将简单介绍地图配色背后的理论。首先是三种基本色系。

稍微调整了一下表达，我觉得地图->讲故事这个解释属于“懂得多余，不懂的还是不懂那种”？此外第一句的翻译意思略有点不同意见，这里 affect the story 应该是 general 的，包括传达给人的信息，故事的重点等等，应该不只是精彩程度。


3.0](http://creativecommons.org/licenses/by/3.0/) license.


Can you check the broker logs for the corresponding disconnect event, log and accompanying stacktrace?


swag


Maybe have the Dr bit in the full_name method, then you can use call:

```
current_user.full_name
```

If not then you could have another method in the User model such as titled_full_name


:ship: it


ack


you don't want a table partial - you just want a _patient partial under the <tbody>. You can then call the following

```
<%= render @patients %>
```

Rails is clever and it will loop through all the patient and render the _patient partial for each one.


Oh okay, I get it. So you're just saying "evaluate this eagerly so the `Boid` structure isn't created with a bunch of uncomputed thunks". Sehr gut.

(of course it's a function, everything is a function)


Introduce a token ``@EMAIL_ARGUMENTS@``?


Ah got it, thank you! I knew there was an explanation in there.


So our boids exist in a 3D space, I guess?


Not sure what the purpose of storing a target point is if we have a velocity vector?


whenever you do a redirect_to in Rails always redirect to the url e.g. new_task_url. It is better for SEO.


I would imagine the wiki is correct and I am wrong since @natefoo wrote the wiki - but he is somehow limiting the number of pulsar jobs - he must be doing that per destination or tag.


this is not very readable. can you think of a better way to do it?


fell


Wanna submit a pr with it?


Assuming this is the 'sight radius' for looking at nearby boids? Is it necessary for this to be associated with each individual?


Yup! (And this change, in conjunction with the rest of the PR, actually does actually fix the more minor of the two problems, namely the stripping of microseconds. But yes this doesn't address the timezone problem, just reimplements the status quo so we have more control over the gradual migration.)


Isn't 'a few' what you want here? http://dictionary.cambridge.org/grammar/british-grammar/little-a-little-few-a-few


yeah....ok


Minor issue with the icon display and handpicked text, sometimes it display a folder, but it's a single download.


Följ gärna kodstandarned.
Borde vara
```
// Send through socket.io to live update preparations.
```
inte
```
//Send through socket.io to live update preparations.
```
https://github.com/jcmandersson/Operation/commit/bb2926a2651c4d0e70f23937e0ed77d015f79c69#diff-fe34f0c8f0cec90a3b40eca51cbada0fR44