although the member's name already give a good description about what they do, please add comment with description


const


Wasn't sure if comments could be removed.  In that case rather than a global array, it would be best to encapsulate in a function that just returns array_key_exists result.  That will open up possibility of converting array to list look up in future if the list becomes large.


Damn, the version of MinGW64 in Ubuntu requires %I64x, and can not handle %llx

Fedora and Ubuntu's mingw64 differ.  I hope there is some version # I can use .


Add Line Break after 7


Quick question are

1. \\ESP\\Model\\Factory
2. \\ESP\\Container

Completely separate inheritance structures?
Separate from the symfony plugin?

I created a test for the other issue
https://github.com/pbyrne84/DynamicReturnTypePluginTestEnvironment/commit/5d95673c145f3f1fdf8a45608b61de96f08678e8

And it seems to work so I am wondering if there is something else going on. Method names always resolve first (design restriction), then inheritance structures come into play so there is room for error some pretty obtuse ways.

Thanks


I've done a bunch more exploration of this now. Some observations:

* Using `cor()` rather than `linreg()` is bad. The R-squared of a regression is much more reliable as an estimate of convergence to a proper number of calls than the correlation.
* After switching to `linreg()` you can safely use a much smaller step size. I'm now setting `α = 1.1`, which gives much smaller final numbers.
* Using a local window correctly is very hard; doing it with simple heuristics (like use the last 25 distinct values on the x-axis) is much worse than including the crazy early values.

My takeaway is that Criterion's solution, which is to not build plans, but instead always do this search and then use the OLS slope as the per-call estimate, is much superior to what I'm doing.

The one problem I see with Criterion's approach is that the bootstrap doesn't account for serious autocorrelation problems, which will produce confidence intervals that are substantially too small.


I'd like something more generic, as I suppose EDF won't be the only app with such a linking. How about:
- replacing appUrl with an object appDetails
- appDetails would contain a field "url", a field "linkTranslationKey" (which would generalize "operations.edf_details") and a field "linkPlainEnglish" (which would generalize "Go to EDF details"). If you've got better naming in mind, feel free to propose any other wording!

This way we could handle other apps in the future.


NOTE, the build on circle 'does' pass, but I would NOT be happy with the end results.


CSS Appearance not supported by: IE (10,11), IE Mobile (10,11)


Add Line Break after 8


<title>OO</title>


`else if (op.appUrl !== null)` would be simpler here


Argh, new global variables! Are you sure they are needed?


I would prefer having these arrays and indexes in the edrv instance structure. Then you don't need initialization, because the edrv instance is set to zero with memset, right?


Here you are doing an unsafe comparison between signed and unsigned data types. Please use the flags in the Makefile when you call the compiler; especially ```-Wall  -Wextra -Wpedantic```


There is still a reference to `router`, which breaks `Link.render` without a `router` in `this.context`. Is this intended or did you just miss it?


Nit: Stray blank line here.


It seems gmp has problems on darwin. http://hydra.nixos.org/build/23183129


I just copied it from your last comments. and this method is not called any way for current SPARK


I would just sleep one second.  Five seconds is a long time for something interactive to wait.  (Imagine if we wanted to show open labors in wtfis, for instance.)


Nit: Stray blank line.


Add Line Break after 9, 10, 11. 12, 13, 15


You may find the [pup](https://github.com/EricChiang/pup) tool useful, as it is designed to run selectors on HTML, not using context insensitive tools like `grep` and `sed`


The fourth point: sold!


Cerinta metodi spune asa: Afiseaza toate proiectele pe care e un angajat ordonate dupa data de start
Eu pot sa vin din afara cu o lista random de Proiecte si sa apelez metoda cu lista ca parametru. Asta nu inseamna ca acea lista reprezinta proiectele unui angajat. Metoda trebuia sa primeasca ca parametru un angajat


Please move that into extra function. I think that it imrpoves readability...


I'm not sure I understand.  You mean that wrt the 'break' above?  If so, yes, I could just return, but I'm sure the compiler will be clever enough to figure that out; I could just also remove the 'default' for that matter.  I could not 'goto out' though as that's a different function ;)


this is incorrect. you should implement a two-stage comparison:
* if the infinity norm of the difference is less than epsilon, return false
* otherwise, return true if ```*this``` is lexicographically less than ```rhs```
* else, return false


oops, GetCurrentPts should not return nopts when going ff or rw.


what?


Also here, this part move into extra function for better readability!


Perhaps this should solve it: https://gmplib.org/list-archives/gmp-bugs/2014-April/003421.html

But someone with darwin access should try it first.


You could also do:

```
return [newArray count] > 0;
```

That way you don't need the flag if and variable.


Really I think this is a message that is shown to end user when they do a bad request


this is done more idiomatically with a ```switch() {}``` statement.


- Add space between "_BROADCAST" and "=="
- Use "INT" in casts


Currently it has to be a map for the insert query, because of https://github.com/ericmj/mongodb/blob/master/lib/mongo.ex#L61-63
That checks explicitly for a map or a list.

I can make a PR to the driver to allow keywords there too, by matching on a list, where first item is a two element tuple.
/cc @ericmj


Voodoo Chronicles will sometimes crash after the 1st 'loading' screen becasue of buggy MsgDiag implementation .

Once pass the loading screen then need to press CROSS to continue .This game is tricky.


The "else if" breaks our coding rules!


old C habits I guess...


ačiū !
merci !
thanks ! ;)


1679-DualDuct (kbenne) - 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)


@skalk The same for the "128" as for the "64" in the ``Mode_transition_control::Allocator``.


This is not good. Keep things simple. Imagine you would have to write the documentation for this method, if it feels awkward, don't do it.
The caller should use `zipUtil.encode(false, io, targetDirName, dataType);` directly. I think then we must export it via metapolatoStandAlone OR you make something like a service for this in here:

```
_p.getZipFromIo = zipUtil.encode;
```

Which is not the worst thing to do in this phase, where we still rather collect requirements than design interfaces ...


maybe ```while (!newPoints.empty())```


@chriscool Yes, will cleanup and push.


I kinda suspect our FITS terminology is pulling back the first (or last) format label and ignoring the other. AFAIK, we've never had a good solution for this in Sufia or ScholarSphere -- can't say if the same is true in ArchiveSphere, but all would benefit from a good common solution! Eagerly following this now.


why not
```coffee
not (item in linters)
```
?


Some thoughts about the formula. I hope this is not so confusing.

Create a class all signs:
namespace QuadraticFormula{
  private string pos_neg_sign;
  private string constant_value;
  private string secondDegree;
  private string firstDegree;
 private string zeroDegree;
  private bool  left_right;
}

public string  Pos_Neg_Sign;
 {
  get{
   return pos_neg_sign;
     }
}
public string Constant_Value{
       get{
   return constant_value;

     }

     }
}

The same for the rest i.e, private string secondDegree
then you can create constructors:

Now you can use all of these as following:
1) you can define a random assignment to the left or right function
2) then:
List<QuadraticFormula>  myQuad= new List<QuadraticFormula>();
myQuad.Add(new QuadraticFormula(paramaters) )

remember to access the List elements:
foreach(QuadraticFormula  temp in myQuad){
    Debug.Log(temp.pos_neg_sign +etc)
}


@skalk Again.


@jgbarah Is it worth to add you query to the document as well? :)


@skalk And again.


@chriscool `git_config(git_tag_config, NULL);` calls `git_tag_config()` as a callback function. `git_tag_config()` needs to manipulate sorting_tails.


In general using `spawn_callback` makes that future separate from the caller (http://www.tornadoweb.org/en/stable/ioloop.html?highlight=spawn_callback#tornado.ioloop.IOLoop.spawn_callback). This means created future bombs out the caller won't be affected. In this case I assumed that _handle_connection wasn't called with the same-- so its tied to something upstream. Basically the goal of this server piece is to separate the specific connection from the rest so they can't break eachother.


PEP8 style requires double blank lines between funcs.


Hi dbongs,

Could you please let me know your preferred time tomorrow and I will reschedule the call. Thanks.

Thanks & Regards
Vivekjyoti Pramanik


why do you use a ```std::set``` and not a ```std::list``` here? Insertion and deletion in a set are logarithmic, but constant in a list


linters is an Array, and `in` is only recommended for objects. I would love to be proven wrong thouhg.


The IOLoop is what executes all the coroutines etc. When you close this object you don't want to stop the ioloop, as that will stop *all* execution on the loop. Since we want a long-running daemon (in general) you just want close to close your specific sockets etc. and unregister them from the ioloop


Here is IMHO better to use the "full path" e.g.:
```
"$ pip install -r django/tests/requirements/py%s.txt" % sys.version_info.major
```

So it's easier to locate ;)


This dependency range includes not-yet-released versions of base, so we don't really know whether it will compile or not. Not sure whether that matters.


LGTM


After a discussion with Ben A :
He absolutely want to move the link in the operation row (no more in details).
And the appUrl should become something like that :
```
appLinks : [{app : String, url : String, linkTranslationKey : String, linkPlainEnglish : String}]
```
Like you see, an operation may have more than 1 link.

Thx for the review


Hi chrisjp,
You have awesome utility application.
One question: How can i check when does admob be loaded?
Thanks! %)


#if HAVE_SELINUX
Please consider using the SELinux setfscreatecon() functionality for the systemctl edit functionality to ensure that unit files and overrides are created with the security context that was specified by the administrators' SELinux security policy.
#endif


@skalk The same here.


This test passes if we configure FITS to recognize RTF as a more specific format than plain text with the following setting inside the "Plain Text" branch in fits_format_tree.xml

```
<branch format="Rich Text Format"/>
```


I think `'--integration-time'` better name for that option.


Can you please share some more details what is done here by adding some more comments into the code!

For me it looks like that you need to translate the memory map seen by the remote processor to the local one. So basically removing the remote shared memory's base address from the kernel buffer and adding the local shared memory's base address. RIght?


国内CDN地址在我这里失效了, 国外的一直都是可以的.


@skalk And here.


Firstly - and most importantly, I was educated in the Florida public school system.
Where misspelling is a recognized constitutional right (also - computers are legally witchcraft).
Also thanks.


May not be needed then :)


Um... it's actually other way around
```
 $ coffee
coffee> a = [1,2,3]
[ 1, 2, 3 ]
coffee> 1 in a
true
coffee> 4 in a
false
coffee> o = {1:1, 2:2, 3:3}
{ '1': 1, '2': 2, '3': 3 }
coffee> 1 in o
false
coffee> '1' in o
false
```

`in` is coffeescript primitive. If not used in for, it expands to `indexOf isnt -1`.


It's just more readable IMO


What is it for? Replacing or not existing blocks?


PEP8 - don't remove blank line.


Did you mean to change this?


:ok:


Whoops, sorry. Fixed it now. Next time can you make an issue instead of a comment? Thanks for the report.


Trail spaces...


Ah, interesting.


It is important that every `StorageDevice` have an instance of `DeviceFormat` as their `.format` attribute. If they have no usable/specific formatting they should use `DeviceFormat` or `getFormat(None)`. If that isn't adequate you can create a new format for internal lvm data.


I take it that it's deliberate that the `rejectionhandled` event is still not cancelable? Need to make sure that's correct in the implementation.


I would personally call the `linter.indexOf(item) is -1` more readable than `not (item in linter)`.
@AsaAyers thoughts?


Change the if ...

    if (tempAddr >= memMapInstance_l.remoteProcSharedMemBaseAddr)
    {
        // Some comment that helps me to understand the code :)
        pBuffer = (void*)( ...);
    }

no else needed because pBuffer is initialized to NULL anyway...


With BCrypt the salt is stored within the password string.


Right, I don't see why it should be; there's no reaction that could take place.


请问如何更改markdown的解析库呢? 现有的不支持`[^footnote]`这样的脚注. 谢谢你!


Please send a PR and add a test in it.


Fixes #347


no need to modify ts files.  the translation are generated using ``scripts/push_ts.sh`` (without commiting here), translated in transifex and pulled back from transifex using ``scripts/pull_ts.sh`` in the nightlies (without commiting anything either).   Only before the release translations get pulled in and committed (using ``scripts/release.pl``)


Precipitation control should always be editable, just not persisted in the attributes of the model if it's a readonly project.  This may be a little unclear because modifications and inputs got grouped together - but they have different behavior. Inputs should always be available for updating to run the model results, but no modifications can be made or changed.


Simplify to:

    val sym =  mixin.implClass.info.nonPrivateDecl(getter.name).suchThat(_.is(Lazy)).symbol

Also: reciever --> receiver, but you don't need the definition, anyway.


Yeah I did, it looked like a mistake to me, since the petition that is being created is actually a hidden petition. However I actually removed the scenario that uses this step definition so can probably just delete this altogether.


There's not actually an assertion in this re-written spec.  Well, nothing other than it not failing because the code raises an error.  The options would be, put the ``expect(Petition).to receive`` line back in (that will fail the test if ``Petition`` does not receive the message - whereas the ``allow(Petition).to receive`` line simply sets up a stubbed response, it doesn't care if it's ever called) or to add something like ``expect(assigns(:petitions)).to eq petitions`` to assert that whatever we do, we get the return value of the setup call.


Add firstOnly parameter to corpus, browse, and compute routes.


Hm, is there any possibility to move the cache invalidation into dualprocshm specific modules? Because in case of single CPU platforms the cache invalidation is already done in edrv.


This should instead be `: ${url:=https://$pkg}`


Or delete the test entirely if we don't think it's useful / is covered by other tests in the same context.


This ought to be right above where we `source` the helpers, but right after the `cd` (so that `.vendor-helpers` can `mkdir -p vendor` for us).


Awesome!


Already been added by @RuiAAPeres ;)


Can you add `.git` on the end of this clone URL just to be extra explicit (and make it more obvious what it is)?


huh yeah, true. My fault, must not have been looking closely. will fix!


@MingcongBai booling the string `loading_build` will always return 1 (format error, in my new impl it is 2), which is the same value as `bool false`.


thanks quite fragile as bash_output return something, so you should at least make fake output like `{ "exit" => 0, "stdout" => "", "stderr" => "" }? Otherwise when someone change code to look at command output if it e.g. failed, broke your tests.


Don't need these curlies


excess curlies


whoops, jumped the gun on this placeholder; should revert to `h$-6` elem selectors until I get around to modularizing.


We do not use react-slider internally anymore.


done


Пока у нас процесс разработки, и мы никуда код не заливаем, то можно изменять предыдущие миграции и пересоздавать базу. Иначе к первому релизу у нас будет несколько сотен миграций


Maybe extra some information from "info" command is a another choice, and it's more clear.
We may need to add some information to output of "info" command.


If it's not used anymore then deleting it is best :)


@thomasvnoort probably.


So that's where the $1 variables happen… I tried finding it but couldn't figure it out.


Ok I will remember that. Thanks.

On Tue, Jun 16, 2015 at 9:03 PM, Erik Ziegler <notifications@github.com>
wrote:

> Whoops, sorry. Fixed it now. Next time can you make an issue instead of a
> comment? Thanks for the report.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/swederik/cornerstoneTools/commit/79c7f1845a2929f1a2c8e8ed2862c20e5613d151#commitcomment-11705718>
> .
>


reack


The format attributes really need to be instances of a `DeviceFormat` subclass. Every block device contains data, which has some format. This is one of the most fundamental concepts of blivet's design.


PYTHON


This paradigm could exhibit a subtle bug in the future. `param` is a class attribute of `dirich`. Similarly, `pi` and `alpha` are class attributes of `param`.

Since they are not *instance* attributes declared in an `__init__` method, changing `dirich`, `pi` or `alpha` could affect *other* instances of `dirich` or `param` (i think). This bug won't rear its head until multiple instances of either class are made - which may not have been tested? Since the parameters are initialised to `None`, this might not matter.  When `param` is instantiated, subsequent assignments to `pi` and `alpha` will override the class attributes with instance attributes (i think). There is also no inheritance (where I think this is particularly dangerous)... I tend to be weary of/avoid class attributes for this reason!

Instead of a struct-like class, a dictionary could be used... Or `param` could be moved outside `dirich` and the parameters could be initialised in an `__init__` method.

There are a bunch of these patterns around this file and [__dist__.py](https://github.com/gabrieag/bayesian-simplex-clustering/blob/master/__dist__.py)


Fun!

This looks pretty good. You may want to exclude test classes in the find_packages call (I think `find_packages(exclude="test")`).

Also when you read in README.rst, you can use this if you'd like
```
import codecs
def readme():
    with codecs.open(
            os.path.join(os.path.dirname(__file__), 'README.rst'),
            encoding='utf-8') as f:
        return f.read()
```
 https://github.com/kylewm/brevity/blob/master/setup.py#L8

The codecs thing is necessary if the environment variable LANG=C and the readme file contains unicode. The path, I don't exactly remember why I needed to add it, something about virtualenvs I think.


The latest versions are released under the "vimeo" package name. Any attempt to install or update the vimeo-api library should be displaying a deprecation notice, pointing you towards that package.

If you don't see the deprecation notice, let me know!


hey @ivanbellalopez! you guys at EyeEm have the latest and most regularly maintained fork of uidevice-extension, would you want to take over ownership of the Cocoapod?


Shouldn't we just ignore everything in shim?


As a matter of convention, there should generally be just one `from __future__` import line.


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

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


Can be just `return`


I seriously doubt that this piece of code was faster to write than using the list of static bloggers and write the htaccess rules in the misc file :)


I know this commit doesn't actually do the failing in CI, but in principle, I think it should be marked as unstable and not failed.


I would put the set ($pedigree = $services...) inside this #if (don't need it otherwise anyway, so no unnecessary work), and move the check for pedigree being defined also inside the if. Otherwise the logic is more convoluted. E.g. we'll get into the else branch which is actually designed to export patient itself even if we selected to export a pedigree but it is not defined


The final solution was in the config files. There is a rake task `fits_conf` that overrides the order in which files are processes that prevents the conflict with RTF files. I'll push a separate commit to document this.


Item not in linter
On Jun 16, 2015 11:34 AM, "Steel Brain" <notifications@github.com> wrote:

> I would personally call the linter.indexOf(item) is -1 more readable than not
> (item in linter).
> @AsaAyers <https://github.com/AsaAyers> thoughts?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AtomLinter/linter-plus/commit/5c2938eb5763de03d4abbfcc755cd59e6067f339#commitcomment-11705741>
> .
>


No need for the "language" IMO ;) "Crystal http/client" if you want, but I'd head for plain "Crystal".


Added file for GA automation


We can lower it of course. It should at least include 4.8 though.


This nesting could be a bit flatter if you do

```js
if (! self._restricted) {
  return selector;
}
```


third index should be 1


I think structure api (or table api) should be another layer. There is no need to change basic source every there. So what do you think? guys


I found a bunch of `numdim` variables in the code (i think there are four occurrences). They don't appear to be defined before they are used - probably a refactoring error? Perhaps they are just supposed to be `dim`?


I committed a hybird here :smiley: https://github.com/AtomLinter/linter-plus/commit/b6c7c3c31f1c4174afd58994c8e1fd1b256dada0


How could an internal lv not have a parent associated with it?


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


@Hiradur : good idea. Actually, we should make a lot more gfx settings configurable so players can tweak their settings.


BIT-TRIP presents Runner 2 , NPUB30733 this game uses write color buffer for every frames and it is good one to test it beside the cube_mrt.elf


+1 since fixed error


Did you intend to combine the effects of case 5: and case 0:?  From what I had read previously, I thought you were modifying the high-end (188 equalization)?


Yannick removed some dependency on dollar Toolbox and enhanced speed


@MingcongBai My shaking hands forgot to strip the slash before opt…


Makes sense - previously {{o}} was used as an argument to another method. I can change this (or if you are already combining the two PRs do you want to do it Jody)?


1679-DualDuct (kbenne) - i386-Windows-7-VisualStudio-12: OK (2177 of 2214 tests passed)

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


why wasn't this declared in the for loop init part below?

Because there's two conditions?


Could the Dirichlet distribution object be replaced by the [scipy](http://docs.scipy.org/doc/scipy/reference/generated/scipy.stats.dirichlet.html#scipy.stats.dirichlet) implementation?


The only option is to move the invalidation to the memmap-noosdual.c as the dualprocshm library is used only for the initialization of the memories and is not involved in mapping the memory for each call.

Also there is no option to specify the size of the buffer to invalidate.


I think the two lines above should go into an `else` block instead of the `try` block.


The structure API is another layer that functions on top of the KV layer.
Perhaps you are looking at what's in branch vivek/methods. Please ignore
that.

Does that answer your question?

On Tue, Jun 16, 2015 at 11:49 AM goroutine <notifications@github.com> wrote:

> I think structure api (or table api) should be another layer. There is no
> need to change basic source every there. So what do you think? guys
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cockroachdb/cockroach/commit/2365a3e3d3eab157fcb1c060fe8c9573d713e144#commitcomment-11706081>
> .
>


Yes was working on this for a while: http://plnkr.co/edit/htsnsfDOjDsAY9g17p26?p=preview

The bottom example illustrates the new issue.

The real problem seems to be the `overflow: hidden` on the `.md-button` class. I am not sure but I think this has no effect and can safely be removed. With that change and back to `display: inline-block` it seems fine. `display: block` works as well, not sure if that might be an option.

Problem #2 is something that has bugged me for a while. Button heights are not consistent in Firefox. `button` is 38px and `a` is 36px as it is supposed to. This can be fixed by removing the `line-height: 36px` from `.md-button` and setting it only for `a.md-button`.

You can test both probelms by uncommenting the CSS.


I wont reject this, but:

convertToShortString is not the right function in the printer. Why not use "%i"?


@lampson0505 Yes, it's been running since last Friday.


It doesn't work for me if I do not use `(intern cmd)`. `cmd` is string and `smex-rank` expects symbol.


These two should go in an `else` block instead of staying in the `try` as well.


Same thing again here.


@lampson0505 It needed some refactoring and at the time (Friday night) I just wanted to start collecting data, so I didn't care about what is where.  I am cleaning up the code now.


The forums have been closed, was just looking for pointers where it might be affecting the addons and drop down menus ect.


кажется для этого можно использовать https://developer.mozilla.org/en-US/docs/Web/API/Element/getBoundingClientRect


why putting it twice ?


@ace20022 entire ff/rw must be broken, right? this should return iRenderPts


Okay. Pretty convincing testcase. I see the problem....


@raix Unless I missed it, I think you are missing tests for modifying `options`. I don't know if passing `options` by reference and allowing all the functions to change it is a good idea. It seems like things could get confusing, especially if some are setting `options.fields` with 0 (exclude) and some with 1 (include).

I think limiting fields is going to be the most common thing people want to do, so we could maybe have the ability to call `this.omitFields([])` in allow.find or deny.find, and NOT pass options. Could also have something like `this.maxLimit(1000)`.

It's still tricky because you can't have fields with 0 and fields with 1 at the same time.


@only-a-ptr @Hiradur So I'll have to redesign the Ingame setting window, because it's a bit too small to add a lot of gfx tweaks and stuff.. :P


Oops, my bad. I thought that `execute-extended-command` was dealing with symbols and not strings.


const


I discovered the __END__ / DATA trick here when I updated some scripts. I think that I have seen its usage more often that the usage of here-document. That is why I used it in this script not because I tought it was the better way.

I asked this question in order to know if you wanted that I modify all the scripts that use __END__ with the here-document. That is all :-)


@SiGe great. Is there any URL to access the result now? Was it the json file you sent me yesterday?


how i contribute with this list?


forget to add a file. fix in 476997032b7e90af8ea13d2ca79824795729b312


The problem is that the script assumes too many things, and whenever we change them we're shooting ourselves in the feet. E.g. HOME and WORKDIR might not be the same post-3.0


Just curious: do we know what in https://github.com/psu-stewardship/archivesphere/tree/develop/fits_conf fixes this? Good stuff!


@Desmaster revertrevertrevertrevert


@SiGe Okay, sure, that is fine. Thx


sp "front-ends"


sp. "chose" -> "choose"


Yes, it felt like 2x. Returning iRenderPts did the trick!


пожалуйста загружайте последние изменения


Replace ```{project_repo}``` with account-financial-tools


@miligraf please check and let me know!!


this diff is removing a "\n" at the end of the file... minor detail


@stephentoub, @mmitche, is this change acceptable?


Hi, I'm the GMime author :-)

I originally named it imap-example.c because the idea was to illustrate how you could stitch back together a message if you downloaded the message parts individually from an IMAP server.

It's still kind of a poorly named example, but oh well, I did that like 10+ years ago :-)


I originally was going this route:

    UIViewController *VC = [ViewController new];
    self.navigationItem.title = (@"%f of %@" ,sender.value, VC.title);

I wanted to create the class from the viewcontroller and use it's title property, but I was getting stuck, so I went this other way.


Crée un nouvel événement pour ça please


Merge with BG.


Same remark as above with link's parameters ```{...}```


I dont understand the meaning of this function. But this line can be moved down out of the loop it seems.


UX is important :)


@Max98 add an advanced tab for things like this, these settings will confuse unexperienced users.


Move it in the while condition


e


Add documentation


You can add 2 tabs to make it nicer (and it will still work perfectly ;))


All of this doc needs to go the way of the dodo. ```patient.toJSON``` should probably return the pedigree, and if not this endpoint needs to become two.


Here to


Next time try to explain in the commit message what bug this fixes. It is not clear to me --- it seems that it will not shuffle the files again, while previously it would reshuffle them again if you asked the test runner to shuffle them?


Precipitation slider should always be editable so that even non owners can run the saved model.  The value should just not be persisted to the scenario attributes.  This may be unclear because "modifications" and "inputs" got grouped together, but they are distinct and have different behaviors.  Anybody will be able run the model with different _inputs_ to see how the land and _modifications_ affect it, even if they do not own it.


Maybe you could do `git_config(git_tag_config, sorting_tails);` and `git_tag_config()` will be passed sorting_tails in its last argument?


K


This function should be in a "details" namespace.


Please remove this, it was taken from the Django template


1679-DualDuct (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2149 of 2214 tests passed)

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


No, this is a flags enum: the values need to be powers of 2.

The change to Any is fine.  Please just add FreeBSD = 8.


Just put everything to false and remove the arrays you can't put to false and work your way through that. I'm not using infiSTAR now because of the lag it causes and floating objects when using a view distance of over 1600m.


@cpsievert Good call.  I think that this should take care of it.


Please update this to https://github.com/openshift/cakephp-ex.git


oops - if I'd realised it was on github i'd have sent a pull request


lastNode is not used somewhere else. Is it really needed?


What's wrong with these? (Re your commit message)


Only include? You raise it so implementation is certainly required.


Oh, not remove, but rename to CAKEPHP_SECRET_KEY as defined in the parameters below.


why ignore the error being returned here?


`<meta>` нужно перенести в `<head>`


Hey Sandor!, do you have an email to drop you some lines?


CakePHP


variables can't have '?' in them, only method names


Sorry about this! I missed it first time. This was actually reshuffling the matched files.


:smile:


finally fixed that memory leak


fail on error? I know it's probably low probability


Did they release a new version with your changes re: ordered maps?


err :)


True!


a few small things, otherwise looks good.

Thanks for your contribution.


It's never used, right ?


Ah, thanks!  Committing a fix right now.


Yes, thanks.


patching this out in 3… 2… 1…


Even if the config value is called "tag.sort", maybe `parse_sort_string()` could be renamed `parse_sorting_string()`.


LGTM :)


Fix is up - once this propagates over to bower, it should be resolved.


You're right! I added it while debugging and forget to clean it up.


For grid search we should set resolution in a smart way as a default, if only budget is given.


horrible spelling


You will need to use clang-format-3.6 on these files.


dont know what do you mean,the sources are already 3.4.42 kernel, i applied the patches to update to 3.4.107


Yes, will do that.


Np thank for your help


Thanks! Done: https://github.com/dotnet/buildtools/pull/180.


Oh yeah! I completely missed out the `void *cb`, thanks :+1:


oh, I see. you added the m_i2p_conn member?
One option might be to push down name resolution into the proxy itself. Similar to how socks5 supports connecting to hostnames. This way perhaps everything in between can stay ignorant about what the name means. (I haven't thought this through btw)


It was used in the ReadMe, and I had to make some changes in order to make it work.


Expediency says this is the best option, but I'd still prefer marking up the text with `<br>` tags somehow...


Wrong again... :)
s/lex/fex/


I updated the patch to use the v1 branches.


This was for ceylon/ceylon.language#670


"all of the following apply" is gramatically correct (not "applies")


( ͡~ ͜ʖ ͡°)


Potentially nothing, but if you could check them that would be good thanks.
I left the url there to make it easier.


indeed


Typo? ("self._vgpp")


- 5? Mean is not n * p for binomial distribution? I don't know if we should check variance too.


1679-DualDuct (kbenne) - 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)


Nice, thanks.
Should I open an issues for the Firefox problem?


Would be nice to get some 4:3 ratio (or less wide ) versions of these for users without widescreens as with https://github.com/RetroPie/RetroPie-Setup/blob/master/supplementary/splashscreens/retropie2015-bw/splashscreen.png


what happens when u call query and ur not connected ?


I would prefer a flags field where padding and lower-case are independent options. But that's a good idea


var_dump + die ?


OK, I did not realize that `last_entry` is a string, I thought it was a regular container. People rarely use iterators on strings, `operator[]` is more common.
You can probably replace `entry_str.at(0)` with `entry_str[0]`, there is a check above that `entry_str` is not empty and `at()` is less efficient.


ack


Removed dans le dernier commit


この状態ではmeasurement_dateが空なのにnew+saveを行おうとしてエラーが発生する．
解決法 -> newに置き換える


@Strahinja we don't control that, you need to tweak your github settings


Nice!! If you want, you can also factorize it with : pythran//pythonic/include/random/random.hpp


createはnew+saveなのでcreateするならここでsaveは不要な気がする．


The only way this would cause the issue is if it hides the player via an invisibility potion instead of "sneaking" the player. However, this appears to not be the case - as this only applies to plugins using the TagAPI, which is now out-of-date (and not installed) http://dev.bukkit.org/bukkit-plugins/paintball_pure_war/pages/documentation/configuration/configuration-v1-2-7/


:+1: I need to write tests for that.


Actually if you see my latest commit, I have removed the dependency on `NugetPorject` completely. Please have a look at Files tab to see the combined changes.


Yes! Based on the last version of the project.


Use a more descriptive test class name.  Something like RegistrationTestCase maybe -- look at the other test files for inspiration if that doesn't strike your fancy.


This whole section should be outdented about four characters.

Also, ``fixtures = ['users']`` is the way we're doing it in other files, so it should probably be consistent.


It only cleans up the .deleteme file if there is one without its corresponding folder. I agree this seems to be side effect of this function. Is the suggestion to change the name? or to remove the deletion logic. Also please note the File.Exists on ln 75 is incorrect (it should be Directory.Exists) .. this is fixed in the 2nd commit (again lets avoid commenting on individual commits and instead use the Files tab to review :) ).


Looking at http://www.unicode.org/repos/cldr-aux/charts/24/summary/sv.html#1415, there appear to be two formatting variants:
* `Months - wide - Formatting Context` has them as all lowercase
* `Months - wide - Standalone Context` has them with a capitial letter

Maybe the wrong one is used?


There is no reason to include ``http://testserver`` in these URLs unless you are matching redirect chains.

Also, try to use ``reverse()`` whenever possible instead of explicit path names.

Move the post options to a separate variable.


I'm not a docker expert, but shouldn't the keys.dat and database be part of separate storage, so that the container can be rebuilt without losing the data?


It should do so; I can add a test for it.


OK, shorter is better. We saw that Ruby uses `"Ruby/#{RUBY_VERSION}"` but we don't have `CRYSTAL_VERSION` yet... but yeah, plain "Crystal" should be better for now.


Yes, definitely open an issue for the Firefox issue.

On Tue, Jun 16, 2015 at 9:18 AM, Peter Leibiger <notifications@github.com>
wrote:

> Nice, thanks.
> Should I open an issues for the Firefox problem?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/angular/material/commit/cc07e63dc5a37b5b220691c3cf8e4b3d08bd2500#commitcomment-11706688>
> .
>


Also, use a consistent naming scheme for the response variable -- ``response`` is a good name.


the original plan has been to use `slot_to` after the for loop has calculated a result. Now I've done it another way, so this is a relic now.


> See this FIXME, probably applies to the regular write() too?

That's a good point. Regular write() should be changed too.

> Also I've noticed the regular write() isn't supported on CompressedSrgbTexture2d but on CompressedTexture2d (and thus write_compressed_data followed it), is it a mistake or there's something that needs to be done differently on sRGB writers?

I think that's a mistake. There are many things that are a bit clumbsy here. Basically my plans were to clean up `TextureAny` (implement everything correctly), then do a cleanup pass on the individual textures.


alterado!!


Do not make blind references to the contents of ``redirect_chain`` -- this will be very hard to figure out six months from now. :-)


`"null"` is wrong from an `R` point of view. When loading the `mzTab` data as an `MSnSet`, it should probably be converted to to `NA`. I am also working on a updated `mzTab` importer, which should deal with these cases. The `FIXME` tag is a reminder to revisit this later on.


Url is not a problem but looking at how the references were done elsewhere I don't think this is quite right. I'm just going to work on them a bit now (including building the docs <- so god knows how long this might take...).


The line items above apply to multiple parts of the tests, not just the particular lines on which they are found.  The tests do pass, and I'm very excited about adding this PR today once these are fixed.  Thanks!


Right, perhaps I was to quick, indeed the order matters here. My mental model was always (in the absence of generics) "number of maximally concrete argument types", but this seems breaks down in the case of overlapping `Union`s. I guess julia doesn't recognize that both definitions are "equally concrete" in the first argument when matching `(Type{Any}, Bool)`


Ideally you'd have a better result, but this can be changed later.


Ño


That's an invalid path. Maybe "." ?


Sorry to be so curious, have you found another solution with the same functionality or you have moved away from this solution (slider) to something more reliable (checkboxes, dropdown, etc.)?


Thanks. I have added your this note in the source code (still commented though) and will add it later, when I am sure that everything works as expected.


What did you think of my earlier comment about the double `dirname()` call vs `os.path.join()`?


removed mac version chromedriver


Yep, 0.1.1 contains the ordered maps.


Done


This line is > 79 chars (run `flake8`)


Thanks man. I did not get to this earlier. Works perfectly for me on Fedora 22.


I disagree with this number.


I searched `$1` in the compiler's specs. There's a reference in [lexer_spec.cr](https://github.com/manastech/crystal/blob/master/spec/compiler/lexer/lexer_spec.cr#L237). From that I went [here](https://github.com/manastech/crystal/blob/master/spec/compiler/lexer/lexer_spec.cr#L115-L119) and then I searched for `GLOBAL_MATCH_DATA_INDEX`, and there's a reference to it in the [parser](https://github.com/manastech/crystal/blob/master/spec/compiler/lexer/lexer_spec.cr#L115-L119) ;-)

(I honestly didn't remember how it was implemented either)


Once we have tested that the integrated code works as expected, we can update this.


BTW it would be better if this discussion was on the origin issue - I will continue there.


Unused. Run `flake8`


nice find. oops


Starting from [here] (https://github.com/ForestryMC/Forestry-Localization/blob/master/assets/forestry/lang/zh_CN.lang#L336), I suspect that Antecer copied them from Forestry localization. Thus, should the title be "add Chinese translation by crafteverywhere" due to crafteverywhere's contribution?


Looks ok


The strategy thread still has the old distribution:
http://steamcommunity.com/groups/MSG2015/discussions/0/598198356167623323/


Can you update the comment here, what are we trying to accomplish


  <Loop>Real\PLAYER\RealPlayHelmetCLoop2d.wav</Loop>
Wouldn't that be
  <Loop>Real\PLAYER\RealPlayHelmetDLoop2d.wav</Loop>
?


You should make the name easier to tie back to the `signalfx` module. The point here is to make debugging easy. `send_thread` could be anything. `SignalFxDatapointSendThread` (or something like that) speaks more.


same here, looks like this if condition is evaluated in multiple places, may be move to one common method


My question was not related to absolute or relative, but about to word ion. The input expects absolute intensities at the peptides level and returns absolute or relative intensities at the protein level. I was confused about ion and thought it would be used to denote individual spectra/PSMs.

Could you clarify the cases where 1 peptide was observed and quantified by several MSMS events. How would these values be combined into one value for the peptides?


Why is this a public method?


Same question


Please review this, @casey-collab


Zero-length arrays can be used in a couple ways to define structs in a flexible way that permits for some polymorphism. For the buffers and tags that you are dealing with, you could probably start out with something like:

```C
typedef struct {
  uint32_t id;
  size_t buf_size;
  uint32_t value_len; /* upper bit reserved for request/response indicator */
  uint32_t request_response_buffer[RTEMS_ZERO_LENGTH_ARRAY];
} bcm2835_mailbox_tag;
````
And then you have,
```C
typedef struct {
  size_t buf_size;
  uint32_t buf_code;
  bcm2835_mailbox_tag tags[RTEMS_ZERO_LENGTH_ARRAY];
} bcm2835_mailbox_buffer;
````

Now you can, for example, do:
```C
bcm2835_mailbox_buffer *b = malloc(sizeof(bcm2835_mailbox_buffer) + sizeof(bcm2835_mailbox_tag) * nt + sizeof(uint32_t) * (nr + 1);
```

Where ```nt``` is the number of tags and ```nr``` is the total number of request/responses across all the tags that will be used in this buffer, plus one uint32_t for the end tag. You can also get fancier with how you allocate these structures, or abstract the allocation behind functions, or define some static allocations using pre-configured layouts for specific kinds of known buffers / tag combinations.


okay


Imagens já estão no google drive. Em casa o programa do git está instalado, então lá fica mais fácil pra eu acabar. Há variações na aceitação das distribuições de um mes para outro, e há umas que estão disponiveis em um mês e não no outro, Então a decisão será quais pegar de outubro e quais de novembro. Penso em usar principalmente novembro, e colocar de outubro só quando necessário. Refazer tudo com dados mais atuais seria um processo bem trabalhoso, não sei se valeria a pena.


@arr2036 could be better move to some *.h? like a centralized header.


Can you not do this for the current world? Is this the right way of doing it?


Other than cleaning up the image, this does not actually result in a smaller image size. The previous layer with all the files still included still has to be downloaded.


I don't see where `_last_post_time` is exposed; you can probably remove it.


Highlighting of the unit tests is broken due to the multiline string on line 140 in that file. I'll try to see fix that as well.


Well, from my experience in the games industry we can pretty much state it as a fact that having to learn Haskell would be a showstopper for the majority that could otherwise be a potential user of LambdaCube, so I really don’t want to change that. Also, ‘learning Haskell’ implies a lot more than just Haskell 98: it’s a skill that includes fluency with the ecosystem of tools around GHC, plus a non-trivial understanding of the implications of using a pure, lazy language.

From the perspective of a tool user, the overlap between Haskell and LambdaCube is not that big if you consider the big picture. In the current formulation, the core language is more or less the same, but everything around it is different: the libraries, the tools, the workflow and the application area.

As for the cons, it feels awkward to me to list those under this question, since they specifically don’t answer it. If you really want such a section, I think it would make more sense to put it under a different question right under it, like ‘What would it take to revive the LambdaCube EDSL efforts anyway?’. At the same time, I’d like to avoid bringing up all the relevant discussion on this page, as it’s a massive scope creep, and the arguments back and forth could go on forever. If we are to add any further details beyond the main points, they should go into a separate article.

Please also note that I added a closing section in f8f4a0c7257a204a5fd236d8cc009d49217dd2c6.


IT will be live in the next release, it's not a filter list which can be added dynamically through `filter-lists.json`.


Can you provide a more detailed description of the issue. Why was it failing to schedule the job when startTime is in the future? How is your change addressing it?


Have you seen in the union station passenger docker image that we have a slimmed down init python file that doesn't have reference to runit? perhaps that's useful to you


getbootsrap.com is giving 404


If the path is an empty string, QDir will be the working directory, and so, it should exist. I do not think that this commit is necessary.


Saving URL... I have been looking for dong.gif forever now.


you mean like this?
![splashscreen](https://cloud.githubusercontent.com/assets/10035308/8188667/6bc22a2a-1414-11e5-8e09-2a4d5bd7a904.png)
or this?
![splashscreen](https://cloud.githubusercontent.com/assets/10035308/8188675/73d2ce68-1414-11e5-842b-f489b412e006.png)


火化——>活化
My typo. 没错这是我手残...


Since the full exception of the current exception context is going to be logged anyway (including its message and stacktrace), it's not really useful to add `with message - %s", e)`:

```python
logging.exception('Posting to SignalFx failed')
```


Technically in here it could be metrics or events, so this message needs tweaking


Can we articulate what the behavior was before and is after this patch?

I'm curious to see what the functional tester finds with this backend and a) current client, b) next-gen client.  I /think/ we shouldn't block on that, other opinions?


Absolutely!


Yep - are they in the repository and I missed them or somewhere else ?


This fixes #48.


```python
getattr(sfx_pbuf, datapoint['metric_type'].upper())
```

Achieves the same thing, but without using an internal method.


OK, though so far I haven't noticed any bugs. The mixture model creates several instances of `gausswish`, and their `param` attributes seem to be updated correctly. What about removing `param`, so that `pi` and `alpha` are attributes of `dirich`?


nope I just created them :P


That would make an array, and we need a dict.


We just use something different internally than this project, thats all.


Yes, you're absolutely right.


Is this what you want, or do you want `raise ValueError('Invalid value ' + value)`?


Having the sid here as well as in TestSigs is confusing


I was wondering where the site went :+1:


Кстати, чего вики не обновляешь? В 1.3 улучшена производительность, а костылей значительное количество ты, как видно, удалил


I would rather add the things you need here to abrt-action-perform-ccpp-analysis and use that script.


You don't use `log_level`, so you can remove it


OK


Aah nice one :) . I prefer this ratio for startup, but then I'm not using 16:9 screens - we should probably offer both, and be clever and choose the correct one on startup or something - but at least we should offer both as a manual choice in the meantime.


This logic seems reverted: shouldn't we bail out with code 0 on the first failure and only return true if all match?


change


Whats the purpose of this class?


Do you know enough to come up with a good rule for doing that? I've found the environment variable hard to track in the past, but we clearly do need to log that information for BLAS perf testing.


In this case we should make sure that no sigs matched I think.


There is no need to outsource the function v_getUrl(), so I reduced the possible dublicates of global functions.


So even if we have a list of sigs, we just check the one in b->sid


I think your suggestion makes sense, the memmap function needs to be extended by the kernel buffer's size. That enables cache invalidation within the memmap module.


People at Reddit are dumb. They thought it was dangerous code, but you are sending GET requests, and not POST requests, and won't even get data from the users.


:shipit: bro!


Good catch – fixed in df6ab1dc00bd9e99df8134db2533978bdc84d741.


This change maybe broke something for me... I now get link errors:

Undefined symbols for architecture x86_64:
  "_restructure_array", referenced from:
      _mirw_hyperslab_normalized in libminc2.a(hyper.o)
      _mirw_hyperslab_icv in libminc2.a(hyper.o)
      _mirw_hyperslab_raw in libminc2.a(hyper.o)

Any clue?


Yeah - cursor keys/history/tab completion. I have all of the cursor stuff working (under linux) I'm just about to add in history and also need to do tab completion. The problem with input() is that I can't also do something else while waiting for the input (like check if the pyboard reset). Input also doesn't deal with tab completion (I really want tab completion on filenames). And readline, which does support tab completion, is only available under linux.

So I'm designing my python readline to be able to process one character at a time.


Not off the top of my head. What options are you using to build?


This is clearly to be implemented and not removed


Yes, potentially. The main differences are:
* Different parameterisation. SciPy uses `alpha` (vector), whereas this one uses `pi` (vector) and `alpha` (scalar).
* SciPy doesn't have a method that evaluates relative entropy, i.e. Kullback-Leibler divergence.

What I could do is create a wrapper class that handles the interface and adds the missing functionality.


yeah it's a good idea. I personally use my widescreen tv as that's all I have at the moment, One way to solve the issue where people wouldn't have to worry about ratios is to have a black background, so it doesn't really matter so much as it will just merge into the black screen, but for those that want a more colourful start up I think offering both options to accomodate different aspect ratios is a good idea. Is there a way to script it to choose one or the other based off of the television someone is using or is that something people would just have to choose manually from the splashscreen menu?


Не хватает guards


Так обычно не делают в h-файлах


Можно по значению возвращать


I think we kind of overloaded the purpose of shims in this project. There's quite a bit of custom code involved in setting up the Backbone/Marionette shims.


There is a way to script it, but there are more important tasks to do, so it might be easier for now just to add these as an option. @petrockblog would you be happy for the two images above to be added too under a different directory name or so ?


Will do.


I think I'm going to replace this whole interface since the compute work done during the plan construction is more than sufficient to give good benchmarking results.


:clap: :shipit:


great


Yes I do actually refer to individual spectra/ PSMs - and the algorithm requires the absolute intensities of each individual PSM and all peptide spectra assigned to a protein are used to estimate the protein ratio. I define the cases where "1 peptide was observed and quantified by several MSMS events" as group of 'redundant peptides' and I even use their distance similiarity as one feature. ( In the manuscipt I refer to 'uniquely' and 'multiply measured (redundant) peptides)
...However, I probably should revisit the manuscript and clarify this more. I might have misleadingly thougt that summarization methods always rely on spectra intensities to infer protein abundances - because how do they usually combine these several (redundant) spectra of one peptide? Publications usually do not comment on this.


CMake, both with the unix makefiles and Xcode generator.  Repros in both cases on two different machines.


müsste die Klasse anstelle einer main nicht nen einfachen Konstruktor haben?


Did you mean to leave the leading space? Same goes for the `substr` two lines below.


macht es Sinn in der Klasse eine Instanz von sich selbst zu erzeugen?


Are you turning off MINC1 support?


I read a bit in jeff's [thesis](https://github.com/JeffBezanson/phdthesis/blob/master/main.pdf) p 72, and it seems `Union(A, B)` and `Union(A, C)` are effectively in a relation of "unknown specificity", so any pair of methods containing these at the same position cannot be disambiguated .

Reading `MsgPack.jl`, it already coerces most int types, so another option might be to use
`retconvert(typ::Union(Type{Any},Type{Integer}), c, val::Union(Int64, Uint64)) = @compat Int(val)`
Or perhaps just
`retconvert(typ::Union(Type{Any},Type{Integer}), c, val::Int64) = val`
as there should be no reason for neovim to emit a Uint64 literal, but I should check it.


This should be `*objects += ...`


This was fixed long time ago, so I guess this change was lost...


Does that mean precipitation should never be saved as well? I could separate "modifications" and "inputs".


No, but it appears LIBMINC_MINC1_SUPPORT is off by default.


ok, fixing this now.


You've gone through this one by hand and confirmed it's degenerate yeah?


Worth including a large non degenerate one? Perhaps just grab rock paper scissors lizard spock?


@NathanW2 , @nyalldawson can you think of a better solution than this to detect if it's a constant `$pi` or a function `pi()` and either needs the brackets `()` to be dumped or not.


> Thanks for the feedback. I like the tool so far. I tried it for the first time on Windows yesterday and wanted to see if I could run it on Linux. It was quite easy to get it running :)

That's awesome. I am really happy that we'll soon have official Linux support thanks to you. :-)

> I think CMake figures this out by itself. I ran it on a 64-bit machine, so it built a 64-bit binary.

Yeah I did some investigating and it decides based on the architecture of your installed kernel. That's good.

> Unfortunately, I'm not sure about this. I think cmake, make and gcc come pre-installed with Ubuntu. I can check this at some point and let you know. Boost should be the only additional requirement. This can be installed with sudo apt-get install libboost-all-dev

Yeah most things come preinstalled but the idea is to make sure nobody's going to be missing any commands they need. :-) Your list sounds correct, though; cmake, make, gcc and libboost-all-dev.

> This was the version I had installed. I'll change it to 1.58

I discovered the reason for this: [Ubuntu is really far behind](http://packages.ubuntu.com/search?keywords=libboost-all-dev). I always build the boost libraries manually, but we might be able to rely on the Ubuntu ones. I read up on Cmake and it will look for the latest version on the user's system that's still compatible with the desired minimum version, so if you write "find_package(Boost 1.46)" and they have 1.55 installed, it will use that instead. All 1.x versions are compatible with each other. So it's actually fine to use Boost 1.46; and if you've confirmed that it's working with that version, then go for it. Initial development was done on Boost 1.47, so I imagine 1.46 should work too. There's been lots of bug fixes since then, but I have no idea what they are. As long as your compiled binary works, I say go for "Boost 1.46" in the cmake file! :-)

> I didn't look much into the code actually. The 'system' subcomponent was listed in build_macosx.sh so I included it here as well. I'll check to see if it's actually required

Ahh yes, sorry about that, turns out we _do_ use the boost_system lib binaries (but not the headers), so I had forgotten that the build script links against "system".

> I tested it on a relatively new Skype account with just two contacts and it ran without problems and I was able to see the conversations. I don't have file transfers, edited messages, /me (as far as I can remember). I'll check emoticons at some point. I can test these as I use them on Skype.

The reason I suggested those in particular is they run a nice selection of Skype features and test a bunch of the parsing at once, especially the boost regex parsing. So I suggest you write a message to someone, then press the uparrow (or right-click) to "edit" it and type something new in there. That takes care of edit messages, and should be output with a little "Edit" pen icon in the exported history. Then send them a file, to see if the output includes the file transfer. Also send them an emoticon. Lastly type /me Something to emote to them.

> I'll make some of the changes you mentioned in the next couple of days and get back to you. Thanks!

Thank you. I appreciate it. Take your time. :-)


Sorry, I rebased this comment out of the PR.


It may make sense to add an autoconf check for this which checks for something like the SHRINKER_NUMA_AWARE flag.


ok, I think it should be fixed.


https://openqa.opensuse.org/tests/68328/modules/bootloader/steps/28 is likely a result of this merge...


Yes this is just a workaround for now and I am investigating a solution that will work on cros and non-cros environments.


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

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


The error message is abit misleading, as the unix socket is not mentioned at all.


Yes the idea is to have same behaviour on both sensors unless there are some serious arguments against that.


std::iota


I sort-of addressed this in my first follow-on commit in the pull request.  I think there are only about 6 or 8 commits of the kernel on which this will not compile.  AFAICT, it will compile correctly at all tagged releases (including stable).  That said, as I indicated, I do think some more, better, etc. autoconf tests are warranted.  I'll try to add one specifically for this case.


I made sure that this is not breaking anything by running all `jxcore` tests (including `mt`/`mt-keep`/`tasks`/packaged/not packaged) for OSX and Windows.


Fixed in 182fa3f.


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


Thanks for the classification. What about peptides with different modifications, different charge states, ... are these also part of the same redundant set?


Pi, A и B хорошо бы объединить в один класс вида HMMModel


Besides that the documentation is shared of both boards.
Changing behaviour between sensors would actually require different documentation for same parameters


You're a saint. :+1:


Не хватает guards


LGTM


Does Traincarts actually work?


I think they should still be saved and started from that saved state, for owners.  This will be more important in future model packages which will have a significant number of inputs that will be difficult to recreate without persisting them.  As far as separating them, we may want to, or even just have two top nodes for `modifications` & `inputs` in the same field since in either case we don't need them to be exposed to database queries or similar.


<!-- PMABOT:SPACE -->
This commit contains trailing whitespace, what is prohibited in phpMyAdmin. Please check our [Developer guidelines](http://wiki.phpmyadmin.net/pma/Developer_guidelines#Indentation) for more information.

Offending files: libraries/Table.class.php


virtual 呢？


1679-DualDuct (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2186 of 2214 tests passed)

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


不 atexit([]{ delete ref; }) ？？


guards


get prefix?


Please review


原子操作呢？


DITTO


秀逗。std::system_error 哭晕在厕所。


Вот эту 0.9 лучше вынести в глобальную константу (а по-хорошему в параметр HMMModel)


就是不懂用 nullptr。


Hello Julien! Thank you for your comments, I updated the pull request accordingly. Please feel free to suggest additional changes :)


不用静态会死星人。


这注释是废话一句。


不把原来的 socket 关掉？


I would actually just do a couchbaseEnvironment() which creates one. If someone wants a shared one, he can override the method ? or is there something I didn't think about


new NMB，unique_ptr 哭了。


Thank you!


错误处理完全不干净，吐槽略。


this is a useful function and needs to be accessible to other classes so I made it `public`.


无话可说。


@mjgiarlo Yes. In fits_conf/fits.xml tool "exiftool.Exiftool" is listed before "jhove.Jhove" and that seems to make a difference. https://github.com/psu-stewardship/archivesphere/blob/develop/fits_conf/fits.xml#L9


居然不是 15。


This fails with a KeyError if kwargs does not contain initial


你家 getter 不带 const ？


whitespace inconsistencies


size_t 会死星人。


I guess the `retvalar()` function is not necessary because sometimes there is a need to make more than 1 query and return more than 1 result.


Finalmente aveva rotto quel suono


@bmsatierf hey Bruno, I made the review and committed my changes (is this the correct workflow?)


std::move() 呢？
好歹也知道 swap() 一下呀。


DITTO


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

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


FacilityTabGridView (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2179 of 2200 tests passed)

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


返回值传值？


Ahahahah
Il 16/Giu/2015 19:22, "Stefano De Grandis" <notifications@github.com> ha
scritto:

> Finalmente aveva rotto quel suono
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/degra91/mobaWksarea/commit/1f7e27dcbdb246aaf9f9fff09f45131a629ea350#commitcomment-11707888>
> .
>


yoda conditionals, i heard you like


滥用 typedef，你这么搞在接口上完全看不出函数参数了。


fixed, thanks.


Thanks for including tests!


虚析构又没了。


:+1:  thanks @pdalpra


Can we define it on class level? I think it would be more reusable if we could just call `typeClass._partialDescriptors`


头文件里不 #include <windows.h> 会死星人?


btw rm recognizez multiple files on one line..


should we do couchbaseCluster and couchbaseBucket?


命名风格不一致。


Why was pk3 chosen as the default format?


couchbaseTranslationService ? I wonder if the prefixes help to distinguish it?


new NMB。exception safety 表示委屈。


This should be handled by ab…


Yep, it is https://nodejs.org/api/events.html#events_emitter_removelistener_event_listener. We can do it in another way, of course.


And ABMK…


Unfortunately this is already the case for `showEditor` too. It would be a nicer solution to let the tableViewDelegate handle this, but I guess for now we can just have TableCell implement `showEditor` and `showReadOnlyEditor` and let those always return false.


unique_ptr 会死。


It's not a good practice to pass all attributes to private variables like this. Attribute values may change during the component's lifecycle, so components need to always get attributes through this.get() to use their current values. Can you remove these lines and change call sites to call this.get() instead of getting the attribute from a variable?


I did it as part of cleanup and merge.


之前说过，用 NMB 静态。


Call it `beyond` then…


Doesn't work for me - getting fatal: Argument "{COMMIT | CHANGE,PATCHSET}" is required


鉴定结论：垃圾一个。Period.


NO! hard-coded abdist!


Not in this instance. The point of naming the pools is so you can track down where in the code the pool was allocated, as the string appears in the output of ``-mMtf`` which is the flag set we use for locating memory leaks. The only change I might make would be to add the file and line, as that can be done by the compiler, and still give us a static/const string.

With ``-mMtf`` when the server exits, it prints out the a tree of all memory which was not freed. That memory is usually the result of memory leaks, as it's not part of the autofree context, and isn't linked to any other contexts.

With the pools named, you know where at least, the request context's pool was allocated, which gives you an idea about the type of request it was.

Using macros would just add an annoying layer of indirection. You'd have to search the source for a macro which defined the string, then search for where the macro was used.


OK, good point.  Changed it back.


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


![screen shot 2015-06-16 at 1 28 04 pm](https://cloud.githubusercontent.com/assets/12915806/8189627/88b479c8-142b-11e5-887c-3e05e44ddfd0.png)


test


Right, we can. Making a PR for it :)


@lileiting that's a good point. Thanks and I committed the fix that you recommended. https://github.com/tanghaibao/jcvi/commit/4c9f1f6defe3854afd2271f2be64ef26907c013d


囗观不识数吱。


I would probably recommend a class attribute instead of a parameter here.


Ah, didn't see the command was on this line... Sorry, and sure.


It´s not enchiladas, you liar!

CHANGE THAT


Published to http://ui.edev.d2l.com/73_aaff2d81d0567ee5db5ad94c285df56410458b32


enabled automatic rocket launching by default


Great @raphaelpor, thanks :+1:


@jamilatta

Neste caso os usuários perdem a possibilidade de buscar pelas variações dos títulos das revistas no índice de revistas. Confirma?


Credit "Humaun Kabir"
 ID "https://github.com/oasiscse"


This should be setting the `enable` attribute to false, instead of an internal variable, so that the attribute always has the correct current value.


Ok, even better :+1:


Same here, this should be setting `enabled` to true.


Why not remove the line?


This needs to change to have a private static OSPlatform constructed around "FREEBSD" which you check against (since the static has been removed).


yes I have


@patois I've tried to recover some symbols that not critical, from libraries and those plugin samples. This version of NTR is weird. I can't even imagine one using a modified FS module from libctru which edits aren't even included in ctrulib repository's history. You're correct - it is easier with what you've already marked out. I renamed some symbols so i can find them easier and added some prototypes.
If interested you can give it a try, or even merge this manually.


@obormot Line 85-86 why you've put parentheses I think, they are redundant...
line 63, why do we need initialization?... for loop handles this...


ok, will do that now


Replace instances of OSPlatform.FreeBSD with OSPlatform.Create("FREEBSD") until the static property is added back.


I think this is redundant


Your fix doesn't work for me anyway. Unfortunately there is no reliable way to control plugins execution order in the same phase. As far as maven-compiler-plugin is declared in maven's root pom it may appear before kotlin compiler anyway in spite of many bugfixes and related issues in JIRA. Even effective pom doesn't reflect actual plugins execution order


This will be moved to the ``__call__`` method and raise an exception if an invalid format is specified. The first format is to be used if ``fig`` or ``holomap`` is ``None``.


It's works?


I'll fix this usage in a following commit, wanted the thing to build and not forget about the since revision. I'm working on using it.


This is a kloogie fix for issue #1.


Thanks! BitHub has sent payment of  $8.58USD for this commit.


More complex than I thought, we can keep it after 0.1.0 as refactoring.


Shouldn't the parent model handle delegations for assigning attributes to partial models? That way we wouldn't need to iterate over partialDescriptors and update each partial model separately, parent model would be the single source of truth.


@davidtkelleher thanks man. I will add the previous/next buttons on the footer.


can no longer preselect value


renamed files for readability.


This change causes OSX Yosemite's built in zlib to not be detected. There's also an error due to cstdint and unordered_map having moved to tr1/cstdint and tr1/unordered_map on OSX so imports involving them fail. Not sure if this is meant to support mac, but just in case.

Discovered while trying to get pocket support for mcedit sorted, we have Win and Linux working but mac is being difficult.


@jamilatta

No índice geral de busca (TW) não será mais recuperado este campo de acrónimo.


what is "tet"?


It seems like this function needs to run when Checkpoint is enabled again. Putting this logic here will only make it work for calls to this `enable` function though. We need to make sure that if someone just sets the `enabled` attribute to true (through `set('enabled', true)`), this behavior will also work.

To do that, you should listen to the `enableChange` attribute, which will trigger whenever the value of the `enable`  attribute changes. There you can run this `refresh` function if the new value is true and so you can remove this call from here.

For example, you'd add the following to init:

```
this.after('enableChange', this.afterEnableChange_)
```

And create the listener:

```
afterEnableChange: function() {
  if (this.get('enable')) {
    this.refresh();
  }
}
```


spelling


You just basicly allowed to dupe bed here. You can join and leave instantly if you use keybind mod.


These getter functions shouldn't be needed. Users can access these values by using the `get` function and passing the name of the attribute to get, for example:

```
checkpoint.get('context');
checkpoint.get('node');
checkpoint.get('offset');
```


Would need some testing with multiple partials but feels like it should work... I'll refactor


@jamilatta

Mesmo caso do ponto abaixo, foi removido do índice de pesquisa geral as variações do título da revista.


It is all considered on spectra level. For each spectra, I assess its seven different feature values: its charge state, modification state, score, ..., and (in case of a redundant spectra): its 'distance to ratios of all spectra referring to the same peptide' (=several MS/MS events for one peptide)  or  (in case of a uniquely measured peptide):' its distance to other "single" peptide spectra which were quantified by only one MS/MS event but assigned to the same protein.

I always relied on evidence.tables / PSM tables for feature and intensity information
I hope this makes it more understandable? otherwise please ask again! Probably it would be best to change all 'peptide' notations in the manuscript to PSM or spectra?


Bro....
What?


You don't need the if clause here, you can always delete the variable instead, which would make the code simpler.


That's funny.


@Saylenty line 63: good point; code health was throwing a warning there.  lines 85-86 you need parens or a ```\```, otherwise it's a SyntaxError


:+1: excellent work @usr110
This is much more elegant than what I suggested


No need for setter functions either, there already is a `set` function for attributes, which can be used as such:

```
checkpoint.set('offset', newOffset);
```


I have been following the videos from a yt channel called "slidenerd". This is the [url](https://github.com/slidenerd/materialtest/commit/a6da4ba042603b635fe342611b01ff882a5148b6) to the commit where the instructor demos a shared element transition. Just for the sake of experimentation, i have declared the profileimage as the shared element, so if i click on it,  it should animate from first activity to the details activity. I have been using a drawable "heart" image for experimentation purpose which is directly specified as "src" in the xml.

I first followed the cliffnotes and then looked in slidenerd's github repo. One thing that i have noticed is that the animations are extremely intermittent. If i click on it close to 8-10 times i will see a weird animation once, but it is definitely not what i expect. I have no idea how to debug this :( I have been testing the code on my phone which has Lollipop 5.1.1

@nesquena


@Saylenty I'll remove it here and in the ethernet decoder. thx


isNearEndGame() is Bool??
spams all abilities now...


@nikolai-b but see my comment in the issue - what's wrong with norwich?!


Fixed in 3ed6d1e80f6948c27104df8f5311070c87d251d7


Should this section be titled "Drupal: Redis is enabled but doesn't have any data" ?


rax retry tests


Well done Platty!


This whole article is about Drupal Redis cache configuration. Now the headers will be inconsistent, and repetitive, and doesn't add to readability.


@mpc-hc/developers: If you agree I will push https://github.com/mpc-hc/mpc-hc/compare/develop...Underground78:e9b765de768771094bad64f2d40d8dfe1120ee2d before we get to many reports about end of playback being broken in the latest nightly.


Makes sense.  Thanks for the clarification (I didn't realize they differed at the low end of the HZ range too).


64 would be a more sane limit IMHO.
What's about performance with large user tables and DB indexes in example?


All attributes should be public, so this shouldn't start with an underscore.


This commit crashes Kodi on startup if OpenSSL-Win32 or OpenSSL-Win64 is installed. I can see in ProcessMonitor that it opens my "C:\Program Files\OpenSSL\bin\openssl.cfg" (it's NOT in path!!) and then crashes Kodi with a message "Unable to initialize. Exiting." It apparently reads the OPENSSL_CONF environment variable and tries to open that file, which leads to the crash. Replacing with the previous libcurl.dll fixes it, as well as deleting the environment variable or renaming the openssl.cfg (which, of course, renders OpenSSL unusable).


@obormot To handle syntax error, you could do like:
elif self.dst.startswith('\x01\x00\x0c\x00\x00') or self.dst.startswith('\x03\x00\x0c\x00\x00')
Moreover, the line length allows.
Meanwhile, good job!


you may consider using addon like ember-bootstrap instead of imporing it manually


One difference I see is that calling DB::getDBConfig('some_db_name', '') will no longer yield any working connection information if the callbacks are being used. From what I can tell, this will not actually affect our code because we always pass either 'read' or 'write' into the getDBConfig method, but it's something we need to be aware of. Passing in an empty string can perhaps be a way of getting the initial connection information and being sure that no extra processing will be executed.

You may want to change the default parameter value for $access in getDBConfigDSN to 'read' instead of ''.


I noticed :+1: , np!! What do you think for moving to a centralized place like a config.h.in?


what is the default namespace?


I think this attribute has a wrong description.


You are right, will fix this.


is the space intended?


Nope, it won't fail, we're only using public interface. The idea of using
`public_send` is to make it clear that we're relying only into public
interface

On Mon, Jun 15, 2015 at 6:02 AM tomas-stefano <notifications@github.com>
wrote:

> Why use the public send? I didn't go to deep on this, but could this
> public send fail?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tomas-stefano/jeckle/commit/5ae904f5b42a5cfbc58de02177386ec114835a4d#commitcomment-11675834>
> .
>


@nikolai-b It seems to be due to the zooming/jumping: zoom into Norwich and nothing happens... Not sure why this is.


Nice work.  Looks like builds are passing now on both x86 and x64. :+1:


Looking at this even more closely I think you'll want to iterate over the online numa nodes with something like this.  I believe those interfaces are available to modules and it ensures your limited to online nodes.  Plus it'll let you don't the `int nid` hunk which is nice.

```
        *objects = 0;
        for_each_online_node_mask(sc->nid)
                *objects += (*shrinker->scan_objects)(shrinker, &sc);
```

We can definitely live with a few specific commits not building as long as everything tagged works.  We might even be able to avoid a full autoconf check and just wrap this in an `#if defined(SHRINKER_NUMA_AWARE)`.  Anyway, I'm sure you'll work up something nice.  :)


Thank you very much!
I have some questions.
does 1000 means " TSAP : 10.00" ? I think that could be Hex or integer nunmber converted from hex.
So, If TSAP is 03.00 --> 0x300(16) --> 768(10) ....
Am I wrong...?


That's the format the mod is in.


In the jinja templates, whenever you define blocks around methods, could you move all these definitions under the constructor? I think that would help with readability/understanding.


Can you add a better description for this class? The description should make the purpose of this class clear so people understand what it's for.

Also, can you explain why this needs to be a separate class, instead of integrating this code with the Checkpoints class?


return entry if entry.name == name

cleaner ^


The title of the article is "Redis as a Caching Backend"

Maybe @nataliejeremy or @rachelwhitton can split out all the Drupal stuff out into another article just on Drupal and this article can be about Redis in general and short and sweet. We already have a separate article for WordPress and Redis...


I meant over pk7.


@ari-gold Please let that be another PR. I'd like to roll this out right away.


Fair enough - I assume that long-term there will be specific tests for both cases so I won't worry about it now.

I'm fairly sure os.hostname only ever returns the hostname part, not the FQDN.

One possible solution to the FQDN problem is to resolve to IP addresses and use those instead.  This isn't necessarily a good answer if the URL is going to be shared since the IP address may differ based on whether you are coming from a public or private network - but it's an option.

Hadoop does something like this.  When I configure a cluster I define internal network IP's (e.g.,10.0.X.X - something not visible to the world) - internal network communication uses those.  Then I set a server setting that says "When talking to the rest of the world, don't return the IP address, return the FQDN portion of the URL".


@sils1297 @Udayan12167 also interesting, that this doesn't change anything in the test outcome.
(Although there are test failing, not ProcessingTest though)...


This access-level optimization is a good one. It will definitely allow for more efficient connection manager.


Well, again, that's the format the mod is in. I don't really think it's wise to have the actual mod releases be pk3 and the git builds default to a completely different format.
Unless you're wondering why I'm using pk3 over pk7 period, in which case Slade can't open/manipulate pk7 builds.


@suzannealdrich Sounds good!


more ember idiomatically you could describe `roadsArray` as computed property:
```javascript
roadsArray: Ember.computed.map('roads.[]', function (road) {
  return {
    strokeColor: this.getStrokeColor(road.get('current_load')),
    path: road.get('path')
  };
})
```

Besides, road array does not seems to be a very nice name for a component property. I would say a road deserve to be an object of it's own.

A class `Road` (which is basically a model) could have a property color, which would replace current `getStrokeColor`.

So generally make it more OO. Models does not have to be persisted.


:cat: (I am still flexing my muscles in art of emoji so some of them might be random)


Oh right, forgot about that.
Ok then, carry on.


Shouldn't this say hello?


Published to http://ui.edev.d2l.com/74_e0b860b2c1e40f019e030d8ba18bf2430b86105b


what is h1 if forgot


This line needs to end with `:`.


The way this is set up, `filename` is no longer defined.


Of course we don't like seeing multiple copies of the same content, be it text or code, but there's no better solution at this point.


Whoops, ignore this. It was added by a mistake and never used.


Consider creating a new model called `Road`. It can describe a class method for fetching roads, so fetching logic will get encapsulated. After all components are UI elements and should not be responsible for data.


h1 is html for the biggest header.


My concern will keeping it separate for such a common case (IE/Edge) is that it's a stumbling block. When I started I grabbed system.js without even considering it'd attempt to pull in a separate file.


Are you kidding me?


@patois I'm sorry but that IDC file really don't apply to the file. When i executed:
```
Can't assign to Segment Register at 0010B440 (T=0000). Trying to recover...
Can't assign to Segment Register at 0010B354 (T=0000). Trying to recover...
Can't assign to Segment Register at 0010B440 (T=0000). Trying to recover...
```
And then i could find some weird part of code..
```
ROM:00106918 ; ---------------------------------------------------------------------------
ROM:0010691C svcSendSyncRequest DCD 0x10B498         ; DATA XREF: sub_1068F4r
ROM:00106920
ROM:00106920 ; =============== S U B R O U T I N E =======================================
ROM:00106920
ROM:00106920
ROM:00106920 sub_106920                              ; CODE XREF: svcAddCodeSegmentp
ROM:00106920                 STMFD   SP!, {R3-R5,LR}
ROM:00106920 ; End of function sub_106920
ROM:00106920
ROM:00106924 ; [00000014 BYTES: COLLAPSED FUNCTION svcGetProcessId. PRESS KEYPAD CTRL-"+" TO EXPAND]
```
The corresponding one is correct:
```
ROM:00106920 ; Result __fastcall srv_RegisterClient(Handle *handleptr)
ROM:00106920 srv_RegisterClient                      ; CODE XREF: initSrv+20p
ROM:00106920                 STMFD   SP!, {R3-R5,LR}
ROM:00106924                 CMP     R0, #0
ROM:00106928                 LDR     R5, =srvHandle
ROM:0010692C                 MOVNE   R5, R0
ROM:00106930                 BL      svcGetThreadCommandBuffer
ROM:00106934                 LDR     R2, =0x10002
ROM:00106938                 MOV     R3, #0x20
ROM:0010693C                 MOV     R4, R0
ROM:00106940                 STMIA   R0, {R2,R3}
ROM:00106944                 LDR     R0, [R5]        ; session
ROM:00106948                 BL      svcSendSyncRequest
ROM:0010694C                 CMP     R0, #0
ROM:00106950                 LDREQ   R0, [R4,#4]
ROM:00106954                 LDMFD   SP!, {R3-R5,PC}
ROM:00106954 ; End of function srv_RegisterClient
```
An IDC file use marks with **offsets** so i don't think it can be done smartly as FLAIR.


Esta linha é responsável pela indexação do resumo nos diversos idiomas e será adicionada novamente no arquivo.


thanks for the link, yeah, makes sense :+1:


Just checking...


Where did you take these keys from?


We use  0R resistor links, only one should be fitted depending on antenna option. Open to further ant tuning suggestions if you have access to rf spectrum analyser


fetching function would take time and current city, so it would be totally decoupled from component state


Crystal is 22pf


this could have some nasty side effects as you change the object that can be use in other places. You should clone it before modifying


this could be a computed property decorated with `.volatile()`


Also, these keys should not be in the repository as this repo is public. We should generate new keys and set them in the `/home/codeeu/coding-events/codeweekeu/settings_local.py` file (which should be kept in sync with `/home/codeeu/deploy/settings_local.py` and out of version control).


They are test keys I create from my Gmail Account. And judging by the fact that it's an Open Repo will need to be changed once we know it works.


@erikschlegel What bug was this addressing? (I'm running into issues with the radio buttons that I'm trying to diagnose and fix.)


sidenote: timestamp is often considered as a single word, thus camelcasing the name does not look right


You had some concerns about this change, but I was not able to understand what bothered you. Can you try to explain again what worries you?


Published to http://ui.edev.d2l.com/75_363ba2376ac4a77a70df0ffed7e7354dd0165a05


I've got a prototype of this working.  I'll clean up and
put up a branch when I have a chance.

+++ Nick Wellnhofer [Jun 15 15 16:28 ]:
>   We should consider to use a binary search instead of a gperf hash for
>   entities. This would reduce the size of the binary considerably.
>
>   —
>   Reply to this email directly or [1]view it on GitHub.
>
>References
>
>   1. https://github.com/jgm/cmark/commit/cdcb65c5a209a1a3be4a1b9e804f5a4292127509#commitcomment-11692021


This feature currently breaks the previous undo stack of any action related to the component that was closed. For example

1. open a component
2. change the frame size
3. close the component
4. press undo twice

An error is thrown on the second undo, because the "change the frame size" action is attached to a no-longer-existent object.

Step 2 above can be replaced by moving the frame, dragging graph axes, and any gear-menu actions currently implemented. All of these are attached to the original object.

I think that undoing by simply creating a new component is problematic. Maybe the original object itself should be stored and then restored? I don't imagine that this will take up any more memory than the current storage pattern used by many undo actions.

Alternatively there could be some system that re-assigns the target of undoable actions to the newly-created object, but I think this would be more complicated and bug-prone.


if you really want to keep it as a utility function you can remove it from component and keeping function statement above component declaration.

You can also make it a pure function - that mean, not depending on state and taking time as an argument.


@Shine- you may just have made me the happiest man alive:) been trying to reproduce and debug for a few hours without getting anywhere. Will be fixed but not reverted as the older curl had another issue.


This is helping for #76 by the way.


+1 to storing them separately. Though this is a big task now since the model running is wired up.


Another extremely-minor comment: I think you forgot to add the action to strings.js?


Okay. Your public key is added to the root user of the server. You can SSH now and deploy it using the instructions I sent you. If you want me to deploy it, let me know.


Yeah, I figured that ;) I thought the Repo was private, but even then, they should be in LocalSettings.
The worry was more wether the backend: social.backends.google.GoogleOAuth2Backend exists. It is a matter of just trying it out and seeing what happens.


:warning: properties with objects and arrays an array "class" are shared among instances. This can cause nasty side effects.


I like :piano:, :drawing:, and  :coding:.


hr


Just verified and it doesn't break the scenario where you use local path or UNC path as your feeds. The new build of DNX works with VS file new project with offline feeds.

We have different code pathes for local/UNC feeds and HTTP feeds: https://github.com/aspnet/dnx/blob/dev/src/Microsoft.Framework.PackageManager/Utils/PackageSourceUtils.cs#L33
So it shouldn't be affected.


this just make no sense. Why not calling then directly on a promise?


Duplicate code, see `KnownMutation.getUnifiedPosition()`


Maybe we shouldn't do this just before release.


Would be cleaner to
1. rename `notFact` to something that does make sense (I know not your fault)
2. make `notFact` a field instead of passing it around


Yes, change to private static OSPlatform s_freeBSD = OSPlatform.Create("FREEBSD"); and use s_freeBSD.


if `setLoadForAllRoads` would create an array of roads you could cache it, and there would be no need for `setLoadForAllRoads` as you could replace roads directly


This field mapping is not necessary, we can avoid setting Id instead.


I think we can get the app name by `mux.Vars(r)["app"]` as well


Your EOL's within the zip are dos/windows style - they should be unix/osx. This is why skin shortcuts is failing for your users - it can't upgrade the .shortcuts files providing your default shortcuts because Python's eval command is choking on the EOLs.


Add separate test method to check for FreeBSD platform. The Xunit.PlatformID should also have a FreeBSD entry now, if we are building and testing for FreeBSD.


TeamCity dddlib :: dddlib [Build 67](http://teamcity.cameronfletcher.com/viewLog.html?buildId=432&buildTypeId=dddlib_Dddlib) is now running


typo: transction


s/directly/immediately ?


in this case `if(cache[cacheId])` woulld do exactly same job. It is just made on assumption that response cannot be falsy (undefined, null, 0, false or empty string)


Just forget about the gradient, the columns matter a whole lot more.


as you have timeout you should probably catch it in some nice way..


Just a thought, is there a reason to use the built-in headers that ship with linux/mac? We already have a local version of zlip in the repository for Win32/WinPhone/Win10. Wouldn't it be better to be consistent and use the same zlib lib across all platforms?


you're sure you need to bind aj_ajax callbacks?


Published to http://ui.edev.d2l.com/76_4b85143c1ccbfef96a18c990de987c287549f711


is this ur commit for the menu assignment, we are going to have to run thru this one while we talk today because its hard to see the exact commit related to only this assingment


TeamCity dddlib :: dddlib [Build 67](http://teamcity.cameronfletcher.com/viewLog.html?buildId=432&buildTypeId=dddlib_Dddlib) outcome was **SUCCESS**
Summary: Tests passed: 177, ignored: 7 Build time: 0:0:0


I imagine this will be alright since upgrades occur at interaction boundaries!


Ok. Thanks. I now added the new secret into the settings_local.py if you want, please give a deploy a try.


Which of these fields receives autofocus?


Overall this seems quite clean and simple. Lets make some issues to track the components which are dependent on `cryptonite` development, though.


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


how about having `updateValue` observer for `value` property:
```javascript
updateValue: Ember.observer('value', function () {
  this.sendAction('setHour', this.get('value'))
}),
actions: {
  setHour: updateValue //not sure if this syntax would work but there is nice chance :-)
}
```


why the + sign is here?


isn't `element.value` already a `value` of the component (because of attributeBindings)?


Actually @mitio let me do the deploy. Now that I am on it anyway.


This #[payload.getId()] should be in a set-payload component, or better, directly in the source attribute for the enricher.


English majors ftw.


> @pnorman - singleton is an internal data structure and not supported for external use. There should be no change in #include directives in client code.

client code (node-mapnik) was using utils.hpp. That bug seems to have been fixed.

Where is it documented that utils.hpp was internal-only?


```js
polishNames: Ember.computed.mapBy('cities', 'polishName')
```
also that is just crappy i18n


`Ember.computed.alias`


why


should be a handlebars helper


looks like view logic, so should be a helper


@certik @isuruf Could you please check if this would work?


Could you make a PR to fix that? Might also be wrong for the fluid container items below.


@monsta
@clefebvre
Can you please fix this for your language at transiflex?


Found it last night after building DX11 with git clean. Reverting DX11 didn't fix it, so it had to have something to do with cleaning / reloading dependencies. :-)


TeamCity dddlib :: dddlib [Build 68](http://teamcity.cameronfletcher.com/viewLog.html?buildId=433&buildTypeId=dddlib_Dddlib) is now running


Fine with me. But since you are on it, put also MONTH and DAY please.

Is YEAR  a AST node as well?


I was just trying to avoid compiling more in a VM than I already had to, wasn't sure what the support state was for it so figured I'd post, reverting this fixes the zlib issue. The tr1 stuff is unrelated.


Не надо класть рабочие файлы студии (или CodeBlocks, не помню чем ты пользуешься) в репозиторий. Удали их и добавь в *.o, *.cbp и пр. в .gitignore файл


My bad, looks like this is it.

https://github.com/jgrant29/address-bloc/blob/menu-continued/controllers/menu_controller.rb <https://github.com/jgrant29/address-bloc/blob/menu-continued/controllers/menu_controller.rb>

Justin

> On Jun 16, 2015, at 12:13 PM, Bobbilee <notifications@github.com> wrote:
>
> is this ur commit for the menu assignment, we are going to have to run thru this one while we talk today because its hard to see the exact commit related to only this assingment
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/jgrant29/address-bloc/commit/186a264b54097efe719fa1e0be74ba974150dfa9#commitcomment-11708908>.
>


@KradekMFC I've finally had time and energy to test this. I implemented the buffer-chunking, which solved the issue I was having with receiving METRICS via chat. I previously just assumed their message might have been malformed, but it was actually split across multiple websocket messages.

With that fixed, I then ran a METRICS test at primetime last weekend. This is how to request all model info via chat instead of via mobj.php:

```
socket.send(new MFCMessage({ Type: MessageType.FCTYPE_MODELGROUP, From: socket.getSessionId(), To: 1, Arg1: 1, Arg2: MFCChatOpt.FCCHAN_PART }));
```

When turning on debug output (showing all contents of all incoming messages), I discovered how MFC deals with this command and why they will *never* risk hitting the "9999" limit even at primetime: They chunk the responses, and send a couple per websocket message.

Like this:
[length][type:FCTYPE_SESSIONSTATE][payload][length][type:FCTYPE_SESSIONSTATE][payload][length][type:FCTYPE_SESSIONSTATE][payload]

Basically, each model is their own individual JSON object carryed inside of their own individual SESSIONSTATE event. That means the message length will never hit 9999 even for chat-metrics. And since those are the longest messages, it seems we never have to worry about any other messages exceeding 9999 bytes either.

One less thing to worry about. ;-)


You don't need this block because ruby wrappers are not tested with C++


TeamCity dddlib :: dddlib [Build 68](http://teamcity.cameronfletcher.com/viewLog.html?buildId=433&buildTypeId=dddlib_Dddlib) outcome was **SUCCESS**
Summary: Tests passed: 177, ignored: 7 Build time: 0:0:0


THis is kind of shitty... but ill take it


What the....


Is there a way to filter out active objects in the queryset, instead of having to loop through all of them after retrieving from the db?


Maybe PR author was inspired by your work :blush: ?


Oh, yea this makes sense, didn't realize throwing an exception here would only happen on the `Map` thread now.


this method is super confusing. why not to:
```js
this.sendAction('changeCity', this.get('currentCity.name'))
```
Warszawa could be just default currentCity


I think it came from the merge, @jrmartin was that something you did in development?


Run the coverity analysis - looks like we eliminated 4 issues and created one new: https://scan.coverity.com/projects/1201


@hijji Sorry it's been too long. Have a nice day.


missing newline before EOF


Hehe, yes I did forget the strings, thanks.

And you're right, this breaks prior undo items that rely on the view components... Storing the view and controller in addition to the model object seemed like it was going to be tough because of the way things get removed from the document and main view. Perhaps we can look up views/controllers by the component model id instead?


OK.


do you know what set OPENSSL_CONF for you? as users are reporting issue I guess it's some relatively popular sotfware?


I am not sure this will work with Travis. I thought you have to pick the environment at the top and be stuck with it, so we pick language `cpp`. I think you'll have to install ruby using `apt-get`.


maybe a filename of `get_python_version.sh` would be better?


looking at actions and reloader I get the impression that this component is doing nothing beside changing state of it's parent. Maybe it should be just a partial?


Since the project language is cpp, you'll have to install ruby manually.


So i've changed it and it seems fine on windows so far, I would like to do a beta on that topic later today or tomorrow. I'll let you know.


I think it's roughly ok, but see our comments above. Travis should be able to pick it up, so you can see what it does.