Las listas de definiciones no se suelen usar en los artículos.
Se suele seguir una estética similar en todo. ¿por qué únicamente los sensores se ponen como definiciones? ¿por qué no los componentes de MUSES?¿por qué no

Por otra parte, sería lo único en negrita en el texto.
Las negritas se usan también muy puntualmente y para señalar cosas muy relevantes, por lo general.

Estaríamos destacando algo sin que se sepa bien la causa.

Esa es mi razón, si quieres que alguien más comente algo y si no se deja como estaba.

@JJ @pacastillo @fergunet?


Looks good :+1:


Do we really need this DS attribute?


Any reason not to all mentions and emoji formatting here too?


I find fact that a header comes after a payload somewhat counter-intuitive. It sounds more like a footer, in fact. Maybe a small ASCII diagram at the top of the file would help clarify how the protocol works. Not too important at the moment, though.


Hi Betty, I think the pull request still currently wants to change jet_l3_correction_x. I thought your comment said there was no need to change it. Are you okay this the way it was once you updated your code? Sorry, I'm just a little confused.


Yeah, move all of the oredict metal blocks over to Chisel Compat... Except for the iron/gold ones :p


Este cambio se realizo con exito, ya puedo ver la tabla perfectamente y el cliente esta feliz


no problem, your modifications were just what I needed (especially ARGB8888 detection) :+1:


No, no, no, I'm happy to have the tau algorithm lines included. I think you misunderstood. I'm sorry to give you that impression. With the removal of the commented lines I am happy with this code.


@embray - I'm also going to try taking this out and make reddening another method at the base extinction model class. Thoughts?


Thanks


Great, thank you for your time! I don't think this redundancy is problematic: `callback_event()` isn't a heavy function, and all this happens admin-side, so... And you are right, a caching plugin could subscribe just to `site.update`, and some other plugin could trigger its own `update_lastmod()` if necessary. Will test it as soon as I can.


@JJ siempre me ha dicho que no se deberían usar listas para definir cosas
si puedes hacerlo de forma "literaria", porque es un paper, no un esquema.
Pero por ejemplo la taxonomía del paper es una lista. Quitad las negritas y
listo.

El 9 de julio de 2015, 18:57, Antonio Mora <notifications@github.com>
escribió:

> Las listas de definiciones no se suelen usar en los artículos.
> Se suele seguir una estética similar en todo. ¿por qué únicamente los
> sensores se ponen como definiciones? ¿por qué no los componentes de
> MUSES?¿por qué no
>
> Por otra parte, sería lo único en negrita en el texto.
> Las negritas se usan también muy puntualmente y para señalar cosas muy
> relevantes, por lo general.
>
> Estaríamos destacando algo sin que se sepa bien la causa.
>
> Esa es mi razón, si quieres que alguien más comente algo y si no se deja
> como estaba.
>
> @JJ <https://github.com/JJ> @pacastillo <https://github.com/pacastillo>
> @fergunet <https://github.com/fergunet>?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/geneura-papers/2014-2015-COMCOM-MUSES_vs_SotA_BYOD_Systems/commit/d0327c13d50c4a41c10f8a9496aa3dd5866de132#commitcomment-12089220>
> .
>

--
Dr. Pablo García Sánchez
University of Granada
http://www.ugr.es/~pablogarcia


Having `activate_check_in_environment` be an array seems like a good idea, it will let us specify more than one environment.

`multi_environment_monitoring` doesn't fit my needs.  Because I have a bunch of checks that are made across all environments, and some checks that are specific to staging, and some that are specific to prod.  So if I'm in staging, I don't want any of the prod checks to be in effect.

I believe this logic used to be where we attach nodes to the service, (I created this from similar code in the commit history).


Feel free to revert it. I am busy with recompiling OpenSSL 1.0.1p and 1.0.2d right now. And, besides that, you are far more familiar with the code and with redis than I am.


Yep, that should be correct. Let me know when you redo the pull request and I will take another look.


This looks like a good spot to add [lodash](https://lodash.com/) as well considering it's now the most depended on [package](https://www.npmjs.com/package/lodash) in npm.


That also works, the only downside is that the exception thrown will always be an `IllegalArgumentException` even if the name is null.


พยายามใช้เป็น % จะดีกว่าครับ เพราะว่าพี่ไม่รู้ว่าจอเค้าขนาดสัดส่วนเป็นยังไง อาจจะทำให้เลื่อนตำแหน่งได้


:ok:


Why do you need this many spaces?


不好意思 好像刚才操作错了。。。 打扰了


Good point.. Hm


OK, anyway thank you so much for your help! I just wrote that because maybe you know the solution which would be good for both systems.
I am going to insert the #ifdef PHP_WIN32 condition here.


I have been told they're working on it. But without there being a public issue tracker (that I can find) or some other way to make it obvious to anyone else with the issue, I felt maybe I'd comment on the part of the .gitignore where a change can serve as a work around. I'm not sure it's discoverable for anyone though.


@halley124 that doesn't change anything, you need to update the `<sort_order>XXX</sort_order>`.


Ok, gracias Fergu.


Desde el punto de vista de estructura del paper el modo como lo había hecho Paloma está mejor: usar definiciones de item. No hay tiempo ya de ponerlo de forma "narrativa", así que por favor ponedlo como definición, como estaba al principio. En general, cuando una persona hace una cosa vamos a suponer que lo hace por una buena razón. No hagamos cambios de este estilo si no son errores gravísimos.


It's a good learning experience and I'm thankful for the history given in the RT.

Sent from my iPhone

> On Jul 9, 2015, at 12:10, Karen Etheridge <notifications@github.com> wrote:
>
> BTW, this patch now exists for core: https://rt.perl.org/Ticket/Display.html?id=125586
>
> —
> Reply to this email directly or view it on GitHub.
>


Cần tạo liên kết tới mục tương ứng trong VarsAndBasicTypes.rst


Para que se vea más claro: una descripción es una descripción. Una lista es simplemente es una lista. Desde el punto de vista semántico (que es el que a mi me gusta) hay que tratar de ajustar los estilos al contenido. Así que no entiendo por qué se ha cambiado description a lista, la verdad.


Ya como veáis, si es mucho lío cambiarlo again, pues nada. Yo lo que quiero es terminarlo :(


(I might have overabused the error handling & error messages in this func)


Hombre, el commit tiene muchas cosas (mal) por lo que no se puede revertir. Si tienes un momento y vuelves a convertir la itemize en descripcion no pasa nada.


Hate, hate, hate this. Much prefer switch/case to if/elseif/else in this situation.


Hmm, looks like that is for statements. This is a property declaration. Does TSLint still say to put it above?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.327](http://teamcity.monogame.net/viewLog.html?buildId=8599&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Lo voy a hacer yo.

Pero JJ, será lo único en negrita del texto.
Porque el estilo de Elsevier de las definiciones es con negrita.


Use of `Authenticable` has been removed.


Hay más cosas que serían más bien definiciones que listas, como las técnicas utilizadas de DM + ML


Right. This would clear out any allocation information someone else might be expecting. It does seem like it would be rare that someone had turned on tracing and then started `MemoryProfiler.report` before collecting their tracing stats.

We should be OK without a clear before our trace. From what I can tell, the trace information is stored on the object itself. So it is taking up memory (which doesn't show in `memsize_of`) for any retained objects. And since we do a full GC first and then check the `allocation_generation` any pre-existing allocation information won't mess up our results. But I'll check on this because I'm not 100% sure that is how it is working.


A mí me da igual, en serio, simplemente no lo entendía y me gustaba así. Si no queda bien por la negrita y no se puede quitar pues no pasa nada.


Yeah. Good call. I didn't like the reset either. Moving this to another class would be good. I may cache the class_name strings too. No point in allocating `"String"` strings thousands of times.


Si lo puedo revertir, es 1 minuto.

Pero por favor, decidme si lo hago o no.


Indeed, `use_*` fns follow the same pattern as `with_*`, thx for clarification :)


Think we'd want to change `button.confirm` in the tests file to `button.js-confirm-modal` to match the core JS addition, right? Thanks for using a `js-` class!


@TJSteveMX you stole my </Stable>
:'(


This can all be removed.  @wontoan and @RamZallan are working on the views for the registration/login form.  Once those are done, I'll merge them into the devise views.


no lo reviertas. Simplemente vuelve a poner los itemize como description.
El resto imagino que se queda igual.

2015-07-09 19:14 GMT+02:00 Antonio Mora <notifications@github.com>:

> Si lo puedo revertir, es 1 minuto.
>
> Pero por favor, decidme si lo hago o no.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/geneura-papers/2014-2015-COMCOM-MUSES_vs_SotA_BYOD_Systems/commit/d0327c13d50c4a41c10f8a9496aa3dd5866de132#commitcomment-12089534>
> .
>

--
JJ


You can avoid tids altogether now with `for &typ in &ns.types { if let Some(Type::Class(ref class)) = typ { ... } }`


They do, I just confirmed them. They are as listed in your format.md.


The param that's missing is the request, not the message, so the error message should probably be something like "You must supply at least one request to redeliver the message to"


Awesome :)

one question:
- we really have two type of yellow color?


We're trying to slowly move away from fixtures to factories using `factory_girl`. I think this could be
```diff
- incoming_message = incoming_messages(:useless_incoming_message)
+ incoming_message = FactoryGirl.create(:incoming_message)
```
and
```diff
- response.should redirect_to admin_request_url(info_requests(:fancy_dog_request))
+ response.should redirect_to admin_request_url(incoming_message.info_request)
```


exchangecalendar-os?

hi thanks for patch.. please note.. its better not to change path. it will give us some confusion.. please change it to exchangecalendar


I left the else part as is because didn't want to break something unwillingly. The if part is something very similar to the example in your link. They're using an array and not a string. Let me know if I misunderstood you, I'll try to clarify.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.327](http://teamcity.monogame.net/viewLog.html?buildId=8599&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:06:30


Always check what scaffold generates for you. We don't use custom helpers at all.


this line cracked me up


# HubPredictor v1.0 beta

**Overview**
HubPredictor is a R runction to run Bayesian Additive Regression Trees (BART) model for predicting chromatin interactions hubs using histone marks information. It can also be used to predict topologically associated domain (TAD) boundaries.

**Systems Requirements**
HubPredictor was written in R languge. It required the R package bartMachine, which is available in http://cran.r-project.org/web/packages/bartMachine/index.html.

**Usage**
Unzip the package. Change the current directory in R to the folder containing the scripts.
HubPredictor.R is the R function to run BART model for predicting chromatin interactions hubs using histone marks information. HubPredictor contains two main steps. Step 1. read TrainingSet data & build BART regression model. Step 2. read TrainingSet data & build BART regression model. It takes three parameters. The first parameter file_in <required> is the input file containing histone marks density for each chromatin anchors. The second parameter file_out <required> is the outpuf file, which includes the prediction information.  The third parameter file_trainingset [optional] is an the dataset for training the BART model. User can define the hubs based on Hi-C dataset accrodding to the method described in Huang, et. al 2015. By default, the code will use the hubs defined based on the Hi-C dataset from the paper Rao et. al 2014 ('TrainingSet/Hub_Rao2014.txt'), which is used in our paper, to build BART model. For TAD boundaries predition, we also provided the  TAD boundaries defined in the Hi-C dataset from the paper Doxin et. al 2012 ('TrainingSet/TADBoundary_Doxin2012.txt'), to build BART model

**Example**
> source("HubPredictor.R")
> HubPredictor(file_in='input.txt',file_out='output.txt',file_trainingset = 'TrainingSet/Hub_Rao2014.txt')

**File Format**
*(1) file_in*
A tab-delimited text file provides histone marks density for each chromatin anchors information, which were used as input for model prediction. More specifically, given a chromatin anchor, we summarized the local pattern for each histone mark by averaging the sequence reads over a 300kb window (about twice the average distance between an anchor and its target site) centered at the anchor location. It can be calculated  using Bedtools coverage.
Format:		chrom	start	end	CTCF	H3K27ac	H3K27me3	H3K36me3	H3K4me1	H3K4me2	H3K4me3	H3K9ac	H3K9me3	H4K20me1
Example:	chr1	201316354	201616354	18.1533	8.25954	15.7391	3.5206	7.97971	6.03226	4.93755	6.72803	3.82303	4.5453

*(2) file_out*
A tab-delimited text file provides the prediciton information, where y_hat_class represents class predictions, and p_hat represents probability predictions.
Format:		chrom	start	end	CTCF	H3K27ac	H3K27me3	H3K36me3	H3K4me1	H3K4me2	H3K4me3	H3K9ac	H3K9me3	H4K20me1	y_hat_class	p_hat
Example:	chr1	201316354	201616354	18.1533	8.25954	15.7391	3.5206	7.97971	6.03226	4.93755	6.72803	3.82303	4.5453	Hub	0.0649590695507929

*(3) file_trainingset*
A tab-delimited text file provides the training dataset to build BART model, which inlcudes histone marks density for each chromatin anchors information (same as file_in) and a column containing the group information.
Format:		chrom	start	end	group	CTCF	H3K27ac	H3K27me3	H3K36me3	H3K4me1	H3K4me2	H3K4me3	H3K9ac	H3K9me3	H4K20me1
Example:	chr1	201134123	201434123	Hub	9.5856	4.44464	10.2766	7.72639	5.97875	5.9307	9.75431	6.8123	2.82555	5.2105
		chr1	97375088	97675088	nonHub	1.39361	1.89214	1.35279	1.94731	3.3451	2.5137	0.557522	1.72618	1.91368	2.29987

**Reference**
J. Huang, E. Marco, L. Pinello, G.C. Yuan. Predicting chromatin organization using histone marks. Genome Biology. 2015.

**Contact**
Jialiang Huang (jhuang at jimmy.harvard.edu or huangjialiangcn at gmail.com)


supply fallback profile repo_name


Relations, not dependencies ;)


skin exchangecalendar fugue chrome/skins/calendar/unix/
skin exchangecalendar  fugue chrome/skins/calendar/win/ os=WINNT
skin exchangecalendar  fugue chrome/skins/calendar/winaero/ os=WINNT osversion>=6
skin exchangecalendar  fugue chrome/skins/calendar/mac/ os=Darwin


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.741.2e3ea91c940a87ee6db076fce298b9fe7ba752f7](http://build.palaso.org/viewLog.html?buildId=59516&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 369, ignored: 3 Build time: 00:04:24


Both lines and msons? Are lines even displayed?
This doesn't look right.


yes, we have :disappointed:
Footer yellow color is different from text highlights


95574472_ip_measure (DavidGoldwasser) - 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)


Good question, I did not consult jsr-310 code, but based this on idea of a separate contribution which only serialized time as timestamp with timezone id (although using slash as separator).
I agree it looks odd, but at the same time there are some benefits.

Another question that is related is whether there is any way to overload `@JsonFormat` to allow per-property inclusion/exclusion of zone id. Earlier I thought that maybe `pattern` should contain zone id, if so, but I don't know if date formats actually support this (they do support tz offset, but zone id)?
OTOH, if timezone id inclusion is supported, then there are even more questions as to whether custom pattern/format should be skipped altogether (current code does not make that easy, there's no way to distinguish, but that could be changed).


Right -- if you're ever in a situation where you've got two different versions of chumpy being loaded, then that's the problem that needs fixing, not something we should be special casing in the code. When you're at your machine hit me up on slack and we'll figure out what's going on with your search path.


Do we really need to create calendar folder here :)

chrome/skins/calendar/unix/
chrome/skins/calendar/win/
chrome/skins/calendar/winaero/
chrome/skins/calendar/mac/


اهلا وسهلا بك في التصميم والبرمجة


Thanks for your nice work....


So now everything will be red? Including the "/"


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.407.906d1df991304ea46024abe54f48da36e54b7097](http://build.palaso.org/viewLog.html?buildId=59515&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 859, ignored: 54 Build time: 00:23:10


Ya, es lo iba a hacer.

Hecho: 1e58a388ccb2fb4fd7972a4a58d0ad1571b00911


Agree, there is probably enough going on in this PR already. :)


No creo que las técnicas de DM+ML haya que ponerlas como description. No las estamos definiendo per se, sino que son "os pasos que sigue MUSES para...". Lo mismo con los módulos y tal, no es una definición de sensores como "un sensor tal es un sensor que hace esto y esto, aquí y en todos sitios".

Pero sí veo lo que comentáis de la negrita, el cambio de estilo y tal. Y como no tengo experiencia pues tampoco tengo mucho más que decir. Me parece bien lo que hagáis.


Can't this be moved down. So the operation is only done when required?


Well, there are more segfaults on *nix, so better just revert it. #ifdef PHP_WIN32's should be avoided as much as possible, now that VC14 is much more compatible with GCC.


Soo @patrickkettner found the issue!

It looks like when you inline everything the HTML gets mucked up.

![image](https://cloud.githubusercontent.com/assets/4303/8601918/39eb6206-2624-11e5-9dc6-47fe358ea929.png)

![image](https://cloud.githubusercontent.com/assets/4303/8601968/7d997b0a-2624-11e5-9e0c-edb57e0e69a6.png)

![image](https://cloud.githubusercontent.com/assets/4303/8601940/565f882c-2624-11e5-9b89-8a846c192fca.png)

IE / MS Edge seems to be having trouble with the messed up HTML (I'll file a bug on them). If you copy the rendered output from Chrome DOM inspector (as it auto corrects the malformed HTML) and use that as the source IE / MS Edge render correctly.


I see, so your team can make dismissive fallacious arguments by authority, offering nonsensical opinions backed up by nothing, and I have to gingerly tiptoe, pleading gently, so that my bug is not rage closed by them.

Ok got it, perhaps I let my annoyance show, sometimes one forgets and says what one thinks of a bad argument, know what I mean?

At no point did you address the substance, which is that it is much better to encode arrays into form field value strings in a way that allows them to be decoded.  Pretty self evident, no?


shit!
thats why Ateek was mad sorry


hmmm, yeaah you're right.
another question rsrs

We have unnecessary color our stylus?


Right. I'll pare this down to just one method called `top_n`. I wanted to keep both around for comparison. This turned out to be an interim step towards calculating the total memsize and obj counts in one call. That was going to be difficult with initial implementation.

I'm also going to look into whether `allocated` and `retained` could be arrays rather than hashes. `allocated` only really needs to be a hash while checking for retained objects.


was my fault.. sorry!


Dunno. Worth revisiting. It's on my to-do list for 0.15, but may be cut.


I like it too!

Sent with Mixmax

On Tue, Jul 7, 2015 at 2:48 PM, Israel Hill < notifications@github.com > wrote:
i like this new shadow however!

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


Thanks @rande - this came in handy :)


XOR or OR?


`evaluate` isn't required to be a staticmethod.  It can be a classmethod or a normal instance method if that helps.

I do plan on a change where additional arguments that aren't parameters can be passed to `evaluate`, which I think would be useful in many areas.  But in the meantime you can definitely use whatever techniques you want to avoid unnecessary repetitive operations.


Depois dessa modificação, algumas tabelas e funcionalidades apresentaram erros, pois a antiga massa de dados que está no banco, poderá apresentar problemas.

*Para corrigir, apenas dê um Flush do banco de dados."


What happened to sed for query?


Thanks!


Incearca sa faci un debug (cu tabelul pe hartie eventual) sa vezi de cate ori se executa ciclul


RadianceRefactor (rpg777) - 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)


I was trying to wait for feedback from Jason on this change before adding it myself. If for some reason he doesn't think it is a good idea then we may be rolling back the functionality.

I'll leave this for now, assuming we will only get some minor nitpicks on the functionality change. This should align with 1.0.1 anyways. If we roll that change back, then we can revert this then easily enough.


IMO that's not really useful, it's only extra code. When do you A. talk to just ElrosGem and B. use sed then?


This is very helpful. Thanks.


I think that's reasonable. If I change it for `send` should I change it for its friends too?


@weaverryan you are welcome ;)


Correct, the script will run on CentOS Linux but it won't installatie Linux itself. Who knows there will be a granubar distro in the future for easy install?


Quizás se pueda modificar un poco esos párrafos para que no aparezcan en negrita esas palabras (forzando la indentación en esos párrafos no aparecerían puntos de listas y tampoco negritas)


+1--Commitin changes and understood Naming convention


I'm not interested in the substance of your bug report given your treatment
of a team that works tirelessly, and have for years, to improve the
framework for all to use free of charge. You were insulting out the gate.
We just don't have room, time, or energy for that here. And that's the end
of it from my side. Best of luck to you.

On Thu, Jul 9, 2015 at 7:25 PM, bughit <notifications@github.com> wrote:

> I see, so your team can make dismissive fallacious arguments by authority,
> offering nonsensical opinions backed up by nothing, and I have to gingerly
> tiptoe, pleading gently, so that my bug is not rage closed by them.
>
> Ok got it, perhaps I let my annoyance show, sometimes one forgets and says
> what one thinks of a bad argument, know what I mean?
>
> At no point did you address the substance, which is that it is much better
> to encode arrays into form field value strings in a way that allows them to
> be decoded. Pretty self evident, no?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rails/rails/commit/19af434840802ca0feb39253241917286467a86e#commitcomment-12089690>
> .
>


Avoid repeated computation.


y u no credit me


Exercitiile sunt corecte :)

Se poate imbunatati coding style-ul:

*Spatii intre operatori:*
$start = 1000;

*Spatiu dupa instructiuni, pentru a le diferentia de executia de functii.* Ex:
A) Instructiune:
for ($i = 0; $i < count($array); $i ++) {
}

B) Apel de functie:
print('text');


+1--replacing insert with set


This isn't route ;)


Exercii corecte


Commented the issue in YouTrack.


Fix google map plotting issues.
Fix color bar value.


Decent :P


+1 camelCaption will be used for table names instead of capital letters


in that case, remove the query logging. logs are only used for sed

On Thu, Jul 9, 2015 at 1:31 PM Jan Dalheimer <notifications@github.com>
wrote:

> IMO that's not really useful, it's only extra code. When do you A. talk to
> just ElrosGem and B. use sed then?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/02JanDal/rubybot/commit/fab6c86c2f664b8bc09409e875faae7c22ddb2b5#commitcomment-12089796>
> .
>


Maybe increase repeated times to 20 or 50 for computing average.


Why isn't `"\\([^.]+\\)?\\(\\.\\.\\.?\\)\\([^.]+\\)?"` enough?


Start to working on plotting grids within analysis area.


+1-- applying changes


Let user draw markers first then draw analysis area.


+1-- proper use of join will be followed onward


Ah well, we can bug him about it next week after SciPy then.


This allocates memory for the whole file's contents, and you don't care about the contents. I suggest using `_, err = io.Copy(ioutil.Discard, reader)` instead.


In what cases do you expect a unix domain socket as a device file? Is that possible?
(or maybe I'm missing something?)


Wonder if `setTimeout` would be a good addition / alternative here to further demonstrate async.


@olivergierke , is there any specific reason for using Collections.synchronizedMap over [ConcurrentHashMap](http://docs.oracle.com/javase/7/docs/api/java/util/concurrent/ConcurrentHashMap.html)?


Foarte bine!


Well Jason already approved this feature in #890 no? My opened PR #895 is only polish/improvement but this Commit tests what was introduced in #890 anyways :)


Good call, lemee give that a shot.

On Thu, Jul 9, 2015 at 1:38 PM, Adam Bray <notifications@github.com> wrote:

> Wonder if setTimeout would be a good addition / alternative here to
> further demonstrate async.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/amaseda/pbj/commit/1a5607549177a787c103088e0c8ccf5761f58d02#commitcomment-12089955>
> .
>


hey


I'll usually just start by calling both implementations on the same input using the `%timeit` magic in IPython.
If the results look interesting I might go on to using the [`timeit`](https://docs.python.org/2/library/timeit.html) module directly to time the two implementations on a bunch of different data of different sizes (a few powers of 10 at least) and, if the results seem interesting, plotting them (with data size on x-axis and time on y-axis)


`utils.removeAllFromArray = function(arr, val) { return R.reject(R.eq(val), arr); }`

;-)


Why is this function needed? You're basically wrapping binary.Read in a function that does exactly the same thing.

Also, if `n != 4` for whatever reason (such as data being partially ready or EOF), you return a nil error.


Use different markers for different channel, but show them in one map.


doesn't look like this guy is used?


Great work!


@jexp should know.


I suspect the Cython version will be faster, but the question is by how much, and to what extent does the data size matter (using Numpy through pure Python code can add noticeable overhead for smaller arrays, but that overhead is usually amortized over larger arrays).


Ah, thanks for pointing that out. I must have missed it. nvm then, clear to go!


Phew, thanks.  Nothing like copy-paste...


TeamCity DnnPackager :: Continuos [Build 15](http://dazinator.ddns.net:81/viewLog.html?buildId=956&buildTypeId=DnnPackager_Continuos&guest=1) is now running


TeamCity DnnPackager :: Continuos [Build 1.0.7](http://dazinator.ddns.net:81/viewLog.html?buildId=956&buildTypeId=DnnPackager_Continuos&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:11


@keul could you please review this? we will probably need to add some tests for `share_link` functions.

can you check if we can get rid of the schema on the URLs?


Yes, much cleaner, thanks


Suggestion for clarification in the question:
Define a function `functionInvoker` that takes a function as an argument (call it `callbackFunc`), and invokes that callback function. Demonstrate using `functionInovker`.


I didn't some test on my local host, I will add some test into repo.
apid stands for Airship Push Identifier. Urban Airship use it through out their documentation.


TeamCity DnnPackager :: Continuos [Build 16](http://dazinator.ddns.net:81/viewLog.html?buildId=957&buildTypeId=DnnPackager_Continuos&guest=1) is now running


TeamCity DnnPackager :: Continuos [Build 1.1.0-unstable.1+1](http://dazinator.ddns.net:81/viewLog.html?buildId=957&buildTypeId=DnnPackager_Continuos&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:07


Why?! :-|


To align the texts. But I think it looks worse with aligned text.


@lmars any particular reason for this change? Originally I had it only responding to errors for the event type we're looking for. With this change, it responds to errors on any events, even if we're not looking for the event type that errored.


It is all lower-case. LGTM.


Pentru lizibilitatea codului recomand sa pui acolade in toate situatiile in care sunt grupuri de instructiuni. Ex:
```
if (cond) {
   echo('test');
}

switch (var) {
   case $value: {
      echo 'test';
      break;
   }
}
```
In rest :+1:


These path changes look like a mistake to me


In this commit, I had copied over `file_paths.md` into `pe_file_paths.md` and then made changes. Do we still want that, or do we think these changes will also be present in FOSS and therefore we should just add the staging directory in the already existing `file_paths.md`?


It should be notice ;]


It's used in creating the custom stop word filter. I just haven't committed
those files yet because they aren't done.

On Thu, Jul 9, 2015 at 10:40 AM, Dmitri G <notifications@github.com> wrote:

> doesn't look like this guy is used?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BranchMetrics/documentation/commit/e2b52715a1ae8f1eaf34973b03e8bc92b0caf9d9#commitcomment-12090000>
> .
>

--
Max Levenson
510.685.9121
max@branchmetrics.io

Find the Branch team at these upcoming events:

July 9 - App Promotion Summit <https://apppromotionsummit.com/LONDON/2015/>
- London

July 22nd - Mobile Growth Hackers
<http://www.meetup.com/Seattle-Mobile-Growth-Hackers-Meetup/events/223360474/>
- Seattle

July 23rd - Mobile Growth Hackers
<http://www.meetup.com/Mobile-Growth-Hackers-Vancouver-BC/events/223663689/>
- Vancouver

July 28th - Mobile Growth Hackers
<http://www.meetup.com/Boston-Mobile-Development-Meetup/events/223361334/>
- Boston

July 29-31 - AnDevCon <http://www.andevcon.com/boston#> - Boston

July 29 - August 1 - RISE
<http://riseconf.com/?gclid=CJKTvfb21cMCFQeRaQodKxYATg> - Hong Kong


Author is @Wolf-DK (I got the email address wrong, sorry for this).


Foarte bine, excelenta ideea cu validarea lungimii sirurilor.


Why not `return string(ImageStreamImageNodeName(n.ImageStreamImage))` like the others?


Buenas tardes Alejandro,

Si si en eso tenes razón, es que comencé haciendo la primera y después hice
un copy paste para comenzar a hacer la segunda función, me quedo con el
mismo nombre.

Saludos

El 9 de julio de 2015, 11:20, Alejandro Oviedo <notifications@github.com>
escribió:

> En general no está bueno declarar funciones con el mismo nombre. Sería
> mejor que una se llame reverseStr y otra reverseArray.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gcuervo/CoderHouse/commit/a9309f130e4aeffa090342dc8e397ca868f1ed30#commitcomment-12085613>
> .
>


Can validation be added to ensure that baseDate is a Javascript Date object?

The following scenario is occurring:
The date format is "MM/dd/yyyy"
Current date is "07/09/2015"
User attempt to modify 09 to 08.
Backspace results in an invalid date of: "07/0/2015"
Entering the full date as "07/08/2015" still shows date validation errors.

When tracking the code, the returned date after the full modification is returned as an invalid date because hours and minutes when instantiating the date in the date parser are both "NaN" because baseDate is still set at the invalid date of "07/0/2015".


Is this required? If so, maybe this should go into videojs-automation?

Also will master still break if I merge this in?


Ah lol, I thought it being a class or static method was part of the model class's magic..


Periods are valid within refnames, just not as flanking characters.  I don't think the whitespace precautions are necessary, though.


Looks like you could use `Array.Empty` directly to spare the JIT compiler from doing inlining.


That probably could have been worded better. It's a array of iovec structs of size `USDF_DGRAM_MAX_SGE` in both cases.


En fait il est recommandé maintenant d'utiliser d'ajouter dans /etc/apt./sources.list.d un fichier snack.list contenant :


EvansHVACObjects (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)


Comentario


This node doesn't have the `Found()` method or a marker on it to indicate that it isn't a normal node.  See https://github.com/openshift/origin/blob/master/pkg/image/graph/nodes/types.go#L25 as a for instance.


These codes only works for one word.
Spec is working for one and two words synonyms.


E corect, dar se poate simplifica.

Nu are rost sa verifici daca `$i != $j` (practic asta faci in linia `if($min != $i) {`), pentru ca intotdeana `$j`e cel putin `$i + 1`.


yes so... I just forgot about this . No worries it didn't break the build also it is tiny repo so not following the gitflow here.


Sorry, accidentally and unintentionally deleted that part. Curious thing it removed exactly the right number of lines so it wouldn't corrupt the script.


apparently don't, but I would have to confirm.


 #[zstack Test Bot Report] P A S S
 Integration Test Result:
Test Summary:
================================================================================
 Test Case Pass Fail TMO Skip
--------------------------------------------------------------------------------
 basic_test:
 suite_setup 1 0 0 0
 basic/test_create_vm 1 0 0 0
 basic/test_stop_vm 1 0 0 0
 basic/test_start_vm 1 0 0 0
 basic/test_reboot_vm 1 0 0 0
 basic/test_add_volume 1 0 0 0
 basic/test_add_vol_to_stopvm 1 0 0 0
 basic/test_add_volume_reboot_vm 1 0 0 0
 basic/test_add_multi_volumes 1 0 0 0
 basic/test_add_volume_negative 1 0 0 0
 basic/test_crt_temp_from_volume 1 0 0 0
 basic/test_crt_vm_with_volume 1 0 0 0
 basic/test_vm_securitygroup 1 0 0 0
 basic/test_create_iso_vm 1 0 0 0
 suite_teardown 1 0 0 0
 virtual_router_test:
 suite_setup 1 0 0 0
 host/test_reconnect_host 0 1 0 0
 novlan/test_create_vm 1 0 0 0
 novlan/test_reboot_vm 1 0 0 0
 novlan/test_ping_vms 0 1 0 0
 host/test_reconnect_host 0 1 0 0
 vlan/test_create_vm 0 1 0 0
 vlan/test_reboot_vm 0 1 0 0
 vlan/test_snat_create 0 1 0 0
 vlan/test_dnat_reboot 1 0 0 0
 vlan/test_ping_vms 0 1 0 0
 vlan/test_create_3vms 0 1 0 0
 vlan/test_start_vm 0 1 0 0
 vlan/test_3l3s_vm 1 0 0 0
 vlan/test_remove_dhcp 1 0 0 0
 vlan/test_2l3_vm 0 1 0 0
 vlan/test_crt_vms_with_limited_ips 0 1 0 0
 vlan/test_crt_vms_with_limited_ips2.0 0 1 0 0
 vlan/test_crt_vms_with_limited_ips2.1 0 1 0 0
 vlan/test_start_vms_simultaneously 1 0 0 0
 vlan/test_crt_static_ip_vm 1 0 0 0
 vlan/test_crt_vm_with_hostname 1 0 0 0
 volume/test_add_volume 1 0 0 0
 volume/test_add_m_vols 1 0 0 0
 volume/test_add_volume_vr 1 0 0 0
 volume/test_crt_vm_volume 1 0 0 0
 volume/test_crt_vm_2_volumes 1 0 0 0
 volume/test_crt_vm_24_volumes 1 0 0 0
 volume/test_volume_ops_on_stopped_vm 1 0 0 0
 volume/test_crt_vm_from_dvt 1 0 0 0
 template_image/test_crt_temp_image 1 0 0 0
 template_image/test_del_temp_image 1 0 0 0
 sg/test_ingress 1 0 0 0
 sg/test_egress 1 0 0 0
 sg/test_icmp 1 0 0 0
 sg/test_udp 1 0 0 0
 sg/test_stopped_vm 1 0 0 0
 sg/test_vm_ops 1 0 0 0
 sg/test_2vm_egress 1 0 0 0
 sg/test_2vm_ingress 1 0 0 0
 sg/test_4vm_complx 1 0 0 0
 sg/test_detach_l3 1 0 0 0
 pf/test_2l3s_pfs 1 0 0 0
 pf/test_attach_pf 1 0 0 0
 pf/test_delete_pf_after_destroy_vm 1 0 0 0
 pf/test_del_vip_before_add_pf_to_vm 1 0 0 0
 pf/test_multi_pfs_sg_tcp 1 0 0 0
 pf/test_multi_rules 1 0 0 0
 pf/test_pf_after_destroy_vm 1 0 0 0
 pf/test_pf_sg_tcp 1 0 0 0
 pf/test_pfs_tcp_udp_same_vip_ports 1 0 0 0
 pf/test_pfs_tcp_with_same_vip 1 0 0 0
 pf/test_tcp 1 0 0 0
 pf/test_udp 1 0 0 0
 pf/test_vm_ops 1 0 0 0
 eip/test_2_eips 1 0 0 0
 eip/test_2l3s_eip_pf 1 0 0 0
 eip/test_2l3s_eips 1 0 0 0
 eip/test_attach_eip 1 0 0 0
 eip/test_create_eip_vm 1 0 0 0
 eip/test_del_vip_before_add_eip_to_vm 1 0 0 0
 eip/test_eip_sg_tcp 1 0 0 0
 eip/test_pf_eip_attach 1 0 0 0
 eip/test_vm_ops 1 0 0 0
 eip/test_vm_ops2 1 0 0 0
 snapshot/test_backup_snapshot 1 0 0 0
 snapshot/test_create_100_snapshots 1 0 0 0
 snapshot/test_create_snapshot 1 0 0 0
 snapshot/test_create_sp_after_delete 1 0 0 0
 snapshot/test_create_image_by_snapshot 1 0 0 0
 snapshot/test_crt_sp_for_both_root_... 1 0 0 0
 snapshot/test_crt_sp_in_stopped_vm2 1 0 0 0
 snapshot/test_crt_sp_in_stopped_vm3 0 0 0 1
 snapshot/test_crt_sp_in_stopped_vm 1 0 0 0
 snapshot/test_delete_snapshot_on_ru... 0 0 0 1
 snapshot/test_crt_sp_in_live_vm 0 0 0 1
 snapshot/test_crt_sp_from_vol_crt_f... 1 0 0 0
 snapshot/test_change_max_sp_depth 1 0 0 0
 snapshot/test_change_max_sp_depth_l... 0 0 0 1
 regression/delete_sg_with_2_attache... 1 0 0 0
 regression/rm_nic_sg_on_stopped_vm 1 0 0 0
 regression/test_data_volume_with_vm... 1 0 0 0
 vr/test_ping_vms_reboot_vr 1 0 0 0
 vr/test_reconnect_vr 1 0 0 0
 vr/test_start_vrs_simultaneously 1 0 0 0
 host/test_reconnect_host 0 1 0 0
 suite_teardown 1 0 0 0
--------------------------------------------------------------------------------
 Test Case Pass Fail TMO Skip
--------------------------------------------------------------------------------
 Total: 102 84 14 0 4
================================================================================


Why the change to `delta_`?


whitespace nitpick here with the `end` (should align with the `do`)


1111 commits!!11!1!!1


*modified


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 124](https://teamcity.labkey.org/viewLog.html?buildId=259976&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:26


you
just was compiling and realised only now - can you move the hook into post build ? (if  there is no other special reason to have it pre-deb)

as if I'm hacking kernel, I'm not running -a install, nor -a deb, just compile run, then cp from build-server.


renaming to adc_power_off would be in line with the current naming for the adc power on function, adc_power_on, and keep naming consistent and descriptive.


Use copyMemory() instead.


Hallo Markus!

Ich habe zwei Fragen:
1. Was bedeutet "Bremse" in Deinem Listing? Die Motoren haben doch keine Bremse? Oder hat das mit dem Motor-Shield zu tun?
2. Warum benutzt Du einen IR-Sensor, statt einem Ultraschall? Ist der IR-Sensor nicht viel anfälliger für Störlicht?


Yeah, I'll change it to `Array.Empty`.

(I had it using `Array.Empty` originally, then changed it to `Enumerable.Empty` before submitting in case it was possible to implement the more "memory efficient" empty enumerable in CoreFX that didn't have the extra enumerator allocation. But since the empty array caches its enumerator, that's no longer a concern.)


## This is a markdown file


@alexmarsev @kasper93 As far as I can tell `GetModuleFileNameEx` works fine.


r+


I don't think pn_data_add() can fail except for out of memory - do you have an example?

It seems to me that pn_data_add() only returns NULL if an allocation failed. The only way for node to become NULL is if pn_data_new() returns NULL and the only way that can happen is is if pn_data_grow() fails.

So, yes, I really think it should be PN_OUT_OF_MEMORY unless you can give me a counter example.


Cool, let's go with it then.


Extend IdentityTransformer instead; it will simplify this class greatly.


The st.st_size was printed immediately before this new print:

dout(10) << __func__ << ": found on disk, size " << st.st_size << dendl;


extract: this.ownerInput.selectedUser.id


Hey @atavism this was breaking regeneration of the ui assets (gulp build) just FYI.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.742.0261e3a13769389c8b55a78221d7bf6142e24a42](http://build.palaso.org/viewLog.html?buildId=59526&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 369, ignored: 3 Build time: 00:03:24


not


add space before />


Probably worth mentioning where this class is coming from?


saw approval from jedmccaleb
at https://github.com/MonsieurNicolas/stellar-core/commit/b73a6d7e0d0e3013b3d8d9f84c7f9c41bba3a992


merging MonsieurNicolas/stellar-core/scpTiming = b73a6d7e into auto


MonsieurNicolas/stellar-core/scpTiming = b73a6d7e merged ok, testing candidate = 8ff1e433


I know that (void) means explicitly ignored.
My point is really that this adds nothing to reading and understanding the code - memmove() by itself explains exactly what it does - it is a subroutine that move a block of memory and has no used return value.

It seems (from your comment) that your purpose in putting ```(void)``` there is educational - to tell developers never to use the return value from memmove(). But I fail to see how this is useful.


I have made a terrible mistake (aand the title too).


Meaningfully guarding loop array accesses outside of their loops would require checking ranges, and to be interesting it'd have to support ranges with non-immediate sizes. Is there a way we can design this so that the implementation can trust the check without having to prove safety from first principles anyway?


Do you think we should turn this into an integration test too? If so, we can automatise that testing as well :)


I wasn't talking about the metal blocks, but whatever :package:


Hi Leon, you asked for review. Here we go:
I don't understand the relationship between the user creation plugin and a monitoring plugin test, which generates a lot of messages solely for testing purposes. Modifying the UI of the usercreation plugin might confuse users. Isn't a unit test better suited or at least put them in the monitorin plugin?


it would be better if this opam file was written in `opam-version: "1.2"`. This will all you to include the `dev-repo:` field to make it easier to pin capnp. For example: `opam pin add capnp --dev-repo`.

Here's an example of a complete opam file in 1.2: https://github.com/ocaml/opam-repository/blob/master/packages/uri/uri.1.9.1/opam#L4

Yours doesn't have to be this complete necessarily but dev-repo, etc. would be nice to have.


These days, with `async`/`await`, I expect people to mostly use `try`/`catch` whichw ill be resilient to both synchronous and asynchronous exceptions. Also, the `Source` subclasses have always been able to synchronously throw exceptions, so I think it's consistent for this one to do so.


good way to readLine from stdin =)


What does that do?


Message is incorrect-- should only say "fixed syntax errors"


This feels really odd.


Yeah, I wasn't at all clear in that comment. What I was trying to say is that you don't need L31-L34 at all. If the `cmd` variable was evaluated to a string like `cat hi.txt | grep hi`, then you could happily pass it to `subprocess.Popen(cmd, shell=True)`. Take a look at the second example in the link.


warlord thats whats getting moved over


Foarte bine!


![Uploading Bard_Square_0.png…]()
![Uploading Yasuo_Square_0.png…]()
![Uploading VelKoz_Square_0.png…]()
![Uploading Sion_Square_0.png…]()
![Uploading RekSai_Square_0.png…]()
![Uploading Kalista_Square_0.png…]()
![Uploading Jinx_Square_0.png…]()
![Uploading Gnar_Square_0.png…]()
![Uploading Ekko_Square_0.png…]()
![Uploading Braum_Square_0.png…]()
![Uploading Azir_Square_0.png…]()


I know


Updated and New Icon


Thanks for improving this!
-Luke


error: unable to create symlink huashan/proprietary/etc/firmware/a300_pfp.fw
error: unable to create symlink huashan/proprietary/etc/firmware/a300_pm4.fw
:wink:


It would be better use the same style of early return here too:
``` C
...
int r = pni_disposition_encode(...);
if (r!=0) return r
...
```


Na, you got that one wrong. The payload is following the header, not the other way around. Sending the payload first would be a dreadful idea. ;)


oh ok. it also doesn't really fit with the code style of the rest of it :-)


you might want to start using ".map". In this example:
val answer = result.fieldSet.map(f = (f._1 -> f._2.toString()) ).toMap


why not private anymore? It would be good to have these classes invisible


looks good!


I apologize for the difference with the indenting in general. In the bimatrix_random_environment file, the only change I was trying to do was to assign the random strategies by doing:
self.row_agents = [ablearn.Agent(random.choice(self.row_strategies)) for k in
				range(self.number_of_row_agents)]
but I messed with the indentation.


Better use the Logger here?!


> Maybe a small ASCII diagram at the top of the file would help clarify how the protocol works.

Documentation will get improved... *a lot*. Just stay tuned. ;)


Combine ```int r``` here with ```r = pni_disposition_encode()`` below. It's always better to declare variables close to or at their first use.


Seems like this can be static.


this is the wrong user id for the analytics pieces. I think what they'll need is: `CandelaLTI::get_external_id_by_userid( $current_user->ID )`


Erm... good catch. :)

Should read `return x ^ y;`, thx.


E corect. Ai putea incepe for-ul de la `$i = 1` ca sa nu ruleze prima iteratie degeaba


Sure, although are the Client changes going in soon? Could bump version when all of that goes in.


did that do the trick?


Chiar daca e o singura instructiune recomand sa folosesti acoladele. Codul e mai lizibil.

```
if (cond) {
   echo('test');
}
```


# HubPredictor v1.0 beta

**Overview**
HubPredictor is a R function to run Bayesian Additive Regression Trees (BART) model for predicting chromatin interactions hubs using histone marks information. It can also be used to predict topologically associated domain (TAD) boundaries.

**Systems Requirements**
HubPredictor was written in R languge. It required the R package bartMachine, which is available in http://cran.r-project.org/web/packages/bartMachine/index.html.

**Usage**
Unzip the package. Change the current directory in R to the folder containing the scripts.
HubPredictor.R is the R function to run BART model for predicting chromatin interactions hubs using histone marks information. HubPredictor contains two main steps. Step 1. read TrainingSet data & build BART regression model. Step 2. read TrainingSet data & build BART regression model. It takes three parameters. The first parameter file_in <required> is the input file containing histone marks density for each chromatin anchors. The second parameter file_out <required> is the outpuf file, which includes the prediction information.  The third parameter file_trainingset [optional] is an the dataset for training the BART model. User can define the hubs based on Hi-C dataset accrodding to the method described in Huang, et. al 2015. By default, the code will use the hubs defined based on the Hi-C dataset from the paper Rao et. al 2014 ('TrainingSet/Hub_Rao2014.txt'), which is used in our paper, to build BART model. For TAD boundaries predition, we also provided the  TAD boundaries defined in the Hi-C dataset from the paper Doxin et. al 2012 ('TrainingSet/TADBoundary_Doxin2012.txt'), to build BART model

**Example**
> source("HubPredictor.R")
> HubPredictor(file_in='input.txt',file_out='output.txt',file_trainingset = 'TrainingSet/Hub_Rao2014.txt')

**File Format**
*(1) file_in*
A tab-delimited text file provides histone marks density for each chromatin anchors information, which were used as input for model prediction. Specifically, given a chromatin anchor, we summarized the local pattern for each histone mark by averaging the sequence reads over a 300kb window (about twice the average distance between an anchor and its target site) centered at the anchor location. It can be calculated  using Bedtools coverage.
Format:		chrom	start	end	CTCF	H3K27ac	H3K27me3	H3K36me3	H3K4me1	H3K4me2	H3K4me3	H3K9ac	H3K9me3	H4K20me1
Example:	chr1	201316354	201616354	18.1533	8.25954	15.7391	3.5206	7.97971	6.03226	4.93755	6.72803	3.82303	4.5453

*(2) file_out*
A tab-delimited text file provides the prediciton information, where y_hat_class represents class predictions, and p_hat represents probability predictions.
Format:		chrom	start	end	CTCF	H3K27ac	H3K27me3	H3K36me3	H3K4me1	H3K4me2	H3K4me3	H3K9ac	H3K9me3	H4K20me1	y_hat_class	p_hat
Example:	chr1	201316354	201616354	18.1533	8.25954	15.7391	3.5206	7.97971	6.03226	4.93755	6.72803	3.82303	4.5453	Hub	0.0649590695507929

*(3) file_trainingset*
A tab-delimited text file provides the training dataset to build BART model, which inlcudes histone marks density for each chromatin anchors information (same as file_in) and a column containing the group information.
Format:		chrom	start	end	group	CTCF	H3K27ac	H3K27me3	H3K36me3	H3K4me1	H3K4me2	H3K4me3	H3K9ac	H3K9me3	H4K20me1
Example:	chr1	201134123	201434123	Hub	9.5856	4.44464	10.2766	7.72639	5.97875	5.9307	9.75431	6.8123	2.82555	5.2105
		chr1	97375088	97675088	nonHub	1.39361	1.89214	1.35279	1.94731	3.3451	2.5137	0.557522	1.72618	1.91368	2.29987

**Reference**
J. Huang, E. Marco, L. Pinello, G.C. Yuan. Predicting chromatin organization using histone marks. Genome Biology. 2015.

**Contact**
Jialiang Huang (jhuang at jimmy.harvard.edu or huangjialiangcn at gmail.com)


and I think you should make another whole element instead of just 2 different data attributes. so like:
`


Why do you still make the mc variables lol


Looks good to me.


@runarberg, you search for the first category with a nonzero probability here, right? Could you please explain why this is the case? Does the same apply to the other end and probability 1? Thanks!


merci!


Initially try to populate from the GetCapabilities: Dimension string, or click a UI button to "load from GetCapabilities", then use stored, user-modifiable values of start, end, interval to generate a range for the Time Manager looping.


Verifica te rog cele trei componente ale lui FOR. A doua, care este conditia de oprire, trebuie sa fie o instructiune conditionala.


:+1: on the quick changes @zachlatta!


Ignore, sent in error. Working on filtering properly.


comment


Oops I forgot to write some comments

I am quite new in working with open source so bare with me and please let me know if I am missing something.

This PullRequest is in connection to the thread "Proposal: Annotating the configuration class", where I originally suggested to have an annotation for specifying which configuration to be used instead of using the hardcoded approach where one has to implement a MockitoConfiguration in the package org.mockito.configuration in order to provide an alternative configuration.

This suggestion is motivated by the discussion in the thread "Proposition: new annotation to inject real objects" where it was discussed to be able to use an alternative injection strategy.

The issues I see in the original approach is as follows:

1) there can only be one "global" configuration when running all the tests.
2) Which configuration is active is "hidding" in the sense that it is not explicit that an alternative configuration is being used.
3) If several jars have a MockitoConfiguration class implemented it can be difficult to reason which configuration is being used.

A counter proposal was being suggested that instead of having an annotation for specifying the active configuration one could have an annotation for specifying the alternative annotation engine to be used.

This PR is implementing the counter proposal

The idea is that one annotates the test case with @MockitoPlugin and the class which the testcase should use as annotation engine,
when initMock attempts to extract the annotation and provide the annotation to the default configuration, which in turn will attempt to instantiate the annotation engine,
and use that in order to populate the fields in the testClass instance.

The specified annotation engine has to implement the AnnotationEngine interface and provide a default constructor and provide an implementation for the process method.

I read in the contribute.md and saw that one should write a fix number in the comment for the commit, however I am unsure if I am privileged to create an issue or it has to be created by somebody else.

br

Meang


Comment should be deleted


Please read code conventions of Java. http://www.oracle.com/technetwork/java/codeconventions-135099.html


$virtual should be a boolean and certainly false by default.


Uhh idk, I just fixed the MP crash :P


Updated


//Shift command is bugged and has their parameters upside down, it says that //shift <direction> amount. But recalls that if you do this, //shift u 50 (invalid direction 5). Please fix.


This comment should be changed to `// VFALCO consider std::shared_mutex`


Actually, it gets the same results, without needing the extra learned parameters (in BN)


Ah. Yes, this would fix the problem.


Please document what the hackery is doing (the reason for it) :)


Did you remove libtime_genoff from trltespr proprietary-files.txt?


Declare variable closer to where it's being used. So, it should be Map<String,Integer> sentenceWordFrequencies = getWordFrequency(sentence)


... and quite common in tags. Could you please remove the whitespace stuff though, I think things would have blown up before we even got here, if the user inserted whitespace (if that's even possible).


Does that work for < 7.1?


Now that I think about it, maybe this dout() should a value less than 10 since this is an unusual error condition.


@feiranwang @netj Is this really the right way to right this parser? I thought MAX, SUM etc. should be in the lexer? The current way essentially means we are trying to do lexing by ourselves?


One overall comment I have is that due to the new error checking there is a very common code shippet:
``` C
int r = function(...);
if (r != 0 ) return r
```
It might be sensible to turn this into a macro to avoid typing it again and again (an inline can't work because of the return), something like:
``` C
#define PN_RETURN_IF_ERROR(x) \
do {\
  int r = (x);\
  // Perhaps this test should really be r < 0?
  if (r!=0) return r;\
} while (0)
```
Used like this:
``` C
PN_RETURN_IF_ERROR(function(...));
```
As this would be widely useful it should go in a widely used private header file, not sure which one though.


Why `NotNull` here? Isn't non-null type sufficient?


Does this repo exist for 5 & 6?


Please avoid one liners. They are a pain to read.
Dolibarr uses PSR-2 coding standard. This means this code should be written:
```php
if ( $virtual==1 AND defined('DOLIBARR_VIRTUAL_TIME_ZONE_OFFSET')) {
    $tzvirtual = DOLIBARR_VIRTUAL_TIME_ZONE_OFFSET * 24 * 60 * 60 ;
} elseif ($virtual==0) {
    $tzvirtual = 0;
}
```

A much better version of it would be:
```php
// With $virtual a boolean
$tzvirtual = 0;
if ( $virtual AND defined('DOLIBARR_VIRTUAL_TIME_ZONE_OFFSET')) {
    $tzvirtual = DOLIBARR_VIRTUAL_TIME_ZONE_OFFSET * 24 * 60 * 60 ;
}
```


If you want it in, please send me a pull request, happy to merge it. Please do update the patch to use tabs for indentation in bash though, for consistency.


Originally that was part of the idea, but for now it's not required--I've worked around the aspects where that might have mattered.  Still, it's important for the *parameters* to use the values passed in to `evaluate()`, and not to look them up on `self`.


Useless comment, that's what git revisions are for.


Please read about java access modifiers and use the appropriate ones here.


جديد


I'd thing the WMS query should be one layer at a time for a specific time, or a time within an interval.   Maybe you would need a queryInterval parameter to differentiate between the nearest time to startTime (TIME=2001-06-07T00:00:00Z&) versus only a data field from October TIME=2001-10-01/2001-10-31&


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/17716/testReport/) for this commit**


if this variable is being used only in one method then it should be local to that method.


Change this line there is a typo.


اتعلم


Remove printing and use assertions.


For goto the flag should also be added for pass2 frees generated below.


RadianceRefactor (rpg777) - i386-Windows-7-VisualStudio-12: OK (2174 of 2202 tests passed)

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


Ok i'ill change it. Wait a moment :)


Exercitiul e corect.

Se poate imbunatati coding style-ul:

**Inainte si dupa operatori se pune spatiu**
```
if ($i % 2 == 0)
```


I don't get it. Why would you need the offset to be stored in the database if it's supposed to be a volatile setting?
Storing it in a temporary manner would be preferable.


s/http/https/


Why the change? Does `"false"` evaluate to `False`?


Ok, the only reason is that `mkknlimg` does it not in the original image, it works like cat.
But no problem, one more lines of code :smile: Will do it later


nit: remove extra whitespace


I like it :)


or does `config.get` return a boolean?


Exercitiile sunt corecte


Merged the files to demonstrate merging?


aaa


dee


This is the best. :+1:


Shouldn't this be ghcFP?


y u no delete


Take a look at libretro/RetroArch@425948c. There was a problem with the udev driver.

Revert this commit and try again, please.

I can confirm now, it broke the config.


# HubPredictor v1.0 beta

**Overview**
HubPredictor is a R function to run Bayesian Additive Regression Trees (BART) model for predicting chromatin interactions hubs using histone marks information. It can also be used to predict topologically associated domain (TAD) boundaries.

**Systems Requirements**
HubPredictor was written in R languge. It required the R package bartMachine, which is available in http://cran.r-project.org/web/packages/bartMachine/index.html.

**Usage**
Unzip the package. Change the current directory in R to the folder containing the scripts.
HubPredictor.R is the R function to run BART model for predicting chromatin interactions hubs using histone marks information. HubPredictor contains two main steps. Step 1. read TrainingSet data & build BART regression model. Step 2. read TrainingSet data & build BART regression model. It takes three parameters. The first parameter file_in <required> is the input file containing histone marks density for each chromatin anchors. The second parameter file_out <required> is the outpuf file, which includes the prediction information.  The third parameter file_trainingset [optional] is an the dataset for training the BART model. User can define the hubs based on Hi-C dataset accrodding to the method described in Huang, et. al 2015. By default, the code will use the hubs defined based on the Hi-C dataset from the paper Rao et. al 2014 ('TrainingSet/Hub_Rao2014.txt'), which is used in our paper, to build BART model. For TAD boundaries predition, we also provided the  TAD boundaries defined in the Hi-C dataset from the paper Doxin et. al 2012 ('TrainingSet/TADBoundary_Doxin2012.txt'), to build BART model

**Example**
> source("HubPredictor.R")
> HubPredictor(file_in='input.txt',file_out='output.txt',file_trainingset = 'TrainingSet/Hub_Rao2014.txt')

**File Format**
*(1) file_in*
A tab-delimited text file provides histone marks density for each chromatin anchors information, which were used as input for model prediction. Specifically, given a chromatin anchor, we summarized the local pattern for each histone mark by averaging the sequence reads over a 300kb window (about twice the average distance between an anchor and its target site) centered at the anchor location. It can be calculated  using Bedtools coverage.
Format:		chrom	start	end	CTCF	H3K27ac	H3K27me3	H3K36me3	H3K4me1	H3K4me2	H3K4me3	H3K9ac	H3K9me3	H4K20me1
Example:	chr1	201316354	201616354	18.1533	8.25954	15.7391	3.5206	7.97971	6.03226	4.93755	6.72803	3.82303	4.5453

*(2) file_out*
A tab-delimited text file provides the prediciton information, where y_hat_class represents class predictions, and p_hat represents probability predictions.
Format:		chrom	start	end	CTCF	H3K27ac	H3K27me3	H3K36me3	H3K4me1	H3K4me2	H3K4me3	H3K9ac	H3K9me3	H4K20me1	y_hat_class	p_hat
Example:	chr1	201316354	201616354	18.1533	8.25954	15.7391	3.5206	7.97971	6.03226	4.93755	6.72803	3.82303	4.5453	Hub	0.0649590695507929

*(3) file_trainingset*
A tab-delimited text file provides the training dataset to build BART model, which inlcudes histone marks density for each chromatin anchors information (same as file_in) and a column containing the group information.
Format:		chrom	start	end	group	CTCF	H3K27ac	H3K27me3	H3K36me3	H3K4me1	H3K4me2	H3K4me3	H3K9ac	H3K9me3	H4K20me1
Example:	chr1	201134123	201434123	Hub	9.5856	4.44464	10.2766	7.72639	5.97875	5.9307	9.75431	6.8123	2.82555	5.2105
		chr1	97375088	97675088	nonHub	1.39361	1.89214	1.35279	1.94731	3.3451	2.5137	0.557522	1.72618	1.91368	2.29987

**Reference**
J. Huang, E. Marco, L. Pinello, G.C. Yuan. Predicting chromatin organization using histone marks. Genome Biology. 2015.

**Contact**
Jialiang Huang (jhuang at jimmy.harvard.edu or huangjialiangcn at gmail.com)


Checks can be _thought of_ as hints, but they would be actual dynamic
checks, and so trap if they fail. They don't have to be trusted by the
implementation; all accesses include an implicit bound check. It's just
that implicit bounds checks guarded by an explicit check are trivially
redundant. Wasm producers could optionally place explicit checks to guard
loops and long sequences of accesses in order to hint that those bounds
checks are trivially redundant.

I think this is the effect that Luke wants to achieve with the compiler
hoisting bounds checks.

Example:
bounds_check(ptr, 4000);  // bounds checks address range [ptr, ptr + 4000],
fails if wraparound
for(i = 0; i < 4000; i++) store(int32add(ptr, i), #88)  // trivially
redundant bounds check since i is in [0,4000)

Example:
bounds_check(ptr, 5)
store(ptr, #99)
store(int32_add(ptr, 1), #100)
store(int32_add(ptr, 2), #101)
store(int32_add(ptr, 3), #102)
store(int32_add(ptr, 4), #103)

On Thu, Jul 9, 2015 at 7:07 PM, Dan Gohman <notifications@github.com> wrote:

> Meaningfully guarding loop array accesses outside of their loops would
> require checking ranges, and to be interesting it'd have to support ranges
> with non-immediate sizes. Is there a way we can design this so that the
> implementation can trust the check without having to prove safety from
> first principles anyway?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/WebAssembly/design/commit/57bdfbe7d8beef86bccf5af4854dfb1f6968651d#commitcomment-12090506>
> .
>


Codul :+1:

Coding style-ul se poate imbunatati:
* acolade dupa IF, chiar daca e o singura instructiune
* spatii inainte si dupa operatori


At this point in the test, we're verifying that button and buttonTwo have the correct data attribute set. Is it worth taking this to the next level and checking on the DOM structure of the buttons post-upgrade (in particular where ripples are being added)? If we want to hold that off for the next set of tests that's fine too.


config.get reads it is a string so we need to convert to boolean. Doing bool(<String>) always returns True, so doing <String> == "true" seemed the simplest way


typo? registration


Looks fine. How about "This will prevent other users from connecting if the host part of their account allows a connection from any (%) host."


i.e., "false" was evaluating to True


Any particular issues with these signatures I can help fix?


just delete den


Add `layout("1.first.col <= 2.first.col")`


s/invalid/unknown/ ?


Oh crap, I read this backwards, as though you had removed the `== "true"`.


:heart:


Explicit errors FTW :)


Nevermind, invalid is fine here.  I was thinking for the case where we're scanning an existing config file.


haha yea... I assume fixing error messages isn't a BC break... We can fix this now if that is the case.


Exerciul e corect, se poate face si fara IF, doar prin alterarea valorii lui $i


Yes, I had been using the same names, until recently was looking at other code and started mimicking this style.


Hahaha, I'm glad to help :+1:


Exerciile sunt corect, as incerca sa le fac si fara IF-uri


It turned out that meteor wasn't put on PATH, because it was now being installed without `sudo`. It's now resolved though.


me gusta el diseño del eventoLoader, pero dentro de cada evento yo ya tengo definido el type del mismo, asique sin tener eclipse al querer obtener un randon no podemos hacer algo como.

EVENTS_MAP.get(Rnd.get(EVENTS_MAP.size()-1)).class.getType()

no se si se puede pero es una alternativa para usar un valor q ya existe


The leading comma comes from [here specifically](https://github.com/google/material-design-lite/blob/master/src/mdlComponentHandler.js#L101-106). I just kind-of assumed it was there for some reason I didn't understand by just reading the code straight. Maybe it cleans stuff up when exploding or something?

Either way, beyond fix-ability until next major.


Kuri, you might want to branch off of develop instead of master.

Master is really only for completed/released code. develop is the active development branch, and will be merged into master only when ready for release


@Sanne, this commit makes the overall build pass successfully finally. Looking into some more details, PR to follow.


Maybe explain the reason is the interpreter for that version isn't installed?


These values are used only here and are not dealt with directly by users. They are abstracted by the boolean `enable` flag in the attributes. That said, I'll add a not to point out how Zabbix defines and uses these values.


That test would belong under Ripples since that is more of a test of that component itself over the componentHandler doing its job.


@julianalucena, @fltiago


I don't think I consciously intended to change the behaviour, happy to change it back.


hi 你好，感谢贡献代码。
请提交到  http://github.com/leancloud/js-realtime-sdk ，这个是我个人的 fork 版本。


Try using App again, use `layout("1.first.col <= 2.first.col")`. Or split out the `Pattern*` to use layout there too?
Be careful, is this one application chain?
```
a b
  c
    d
  e
```


Will it be merged into 2.8 soon in order to fix it in there too ?


See PADT comment above. Same applies here


Could you keep the rsearch stuff in there, I plan to use it to automate the slideshow


Add as a comment `# Fall back to building the entire environment`?


Nice. Still being installed without sudo, I hope?


Anahita 4.1.3 Birth Release

how do i install anahita


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


If I may make a suggestion, make the returnWailaHead a function from a WailaDataProvider interface, that way we can just implement that interface on any Tile, not just the cores


I will remove this part and just keep the text-content fixtures out of this repo.


yeah, `export PATH`.


Don't forget to add the `defaults` when you migrate from 0.2.7.


better to call it "eligible_source_row" than "valid_cut" -- both for the function name and the error message


it should probably be "number_of_cuts" (plural)


oh nvm, this is the name of the test itself. Updating now and ammending commit.


Also changed the for loop indexed over the array of arguments to a foreach loop. Function is identical; just demonstrating a different looping type.


There are pros and cons of merging lexical analysis with parsing itself.  Other than the two downsides below I think it's better to not have a separate lexer and keep the grammar in one place.
* Parsing at the level of characters is obviously slower than doing at token level.  Unless we need to parse huge DDlog programs at high rates I don't think this matters.
* The error messages can be misleading since parser can stop at the middle of a word, not showing the rest of it.  However, because character sequences are not predetermined as a fixed set of tokens, we have the flexibility of freely using token and their prefixes in various ways.

There seems to be [debates over this issue](http://programmers.stackexchange.com/q/128888). :)


Why do we need this? Can't we just override methods in the anonymous classes?


Вынеси определение факторов в отдельную функцию.


Please add some more detail so to the error message so that user knows what wrong and with which file.  My Ruby knowledge is pretty slim, but something like this should work:
```
rescue => err
    printf "ERROR: Dataset file %s could not be read: %s\n", file, err.message
end
```

Also, you can test it by removing read permissions from one of the files with a command like this: `chmod ugo-r someDataFileName`. When you're done testing the change, you can restore permissions with `git checkout -- someDataFileName`


fast-forwarding master to auto = 8ff1e433


Having user modifiable values for start and end already exists for raster layers at the UI level


@arturoc it's either sudo-less and container builds (where you have a cache), or sudo required, but no cache available. I think with our install_deps scripts we don't get around sudo (and we should keep testing those, too)


Not sure if `npm install` works in this context of `_shared/install_ff_nightly.sh`.


and **has** no formatting errors, since "your list" is singular.  :p

I also suggest: **gramatically** correct


93469234-DemandResponse2 (vidanovic) - x86_64-MacOS-10.9-clang: OK (1684 of 1687 tests passed)

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


I try something.
I can not do without the "-os". If I would do it, then the "skin exchangecalendar fugue" lose its function. The whole calendar is then without working css.
What can i do is, put the os-styles behind the fugue-Path. The crome.manifest would look like this:
skin exchangecalendar fugue chrome/skins/fugue/
skin exchangecalendar-os fugue chrome/skins/fugue/unix/
skin exchangecalendar-os fugue chrome/skins/fugue/win/ os=WINNT
skin exchangecalendar-os fugue chrome/skins/fugue/winaero/ os=WINNT osversion>=6
skin exchangecalendar-os fugue chrome/skins/fugue/mac/ os=Darwin

This would work. Should i do this?


saw approval from jedmccaleb
at https://github.com/MonsieurNicolas/stellar-core/commit/559fda878892808f089a47c8213b9ca5a0e9b8a3


merging MonsieurNicolas/stellar-core/txMetaUpdate = 559fda87 into auto


MonsieurNicolas/stellar-core/txMetaUpdate = 559fda87 merged ok, testing candidate = cedf2f9b


переменные капсом не нужны, тем более в аргументах функции


rename to "slightlyBetterThanNothingIdentity"?  I didn't second guess the existing variable name, so I'm going to have to trust it.


> That test would belong under Ripples since that is more of a test of that component itself over the componentHandler doing its job.

Sgtm in the name of keeping test responsibilities clear. In that light, this test lgtm.


Switched to issue #32


OK, I've updated it.


Should be ACE_player. That ensures that the progress bar is still shown if the player does the action while controlling a unit through Zeus or unit switching.

I also recommend reversing the if, to get cleaner code:

```sqf
if (_caller != ACE_player) exitWith {};
```


Problema asta am inceput-o in timpul cursului si am facut impreuna structura array-ului.

Pentru a putea exersa elementele din curs, recomand sa o mai faci o data (toate exercitiile din ea) pornind de la urmatorul sir (am pus doar o persoana, adauga-le tu pe celelalte 2):
```
$friends = array(
  array(
    'first_name' => 'John',
    'last_name' => 'Doe',
    'ticket' => array (12, 44, 1, 8, 22, 6)
  ),
  [..]
);
```


Purpose

 This allows users to be able to get log information. This will also allow more interesting use cases for the API and be able to use the v2 api for the project summary page.

Changes

Added Logs link in node details "http://localhost:8000/v2/nodes/3ybtm/logs/"
![screen shot 2015-07-01 at 9 44 47 am](https://cloud.githubusercontent.com/assets/9525062/8603860/c8af95f6-2648-11e5-8b0d-cb4b7161b114.png)

Node Log List

![screen shot 2015-06-30 at 4 00 23 pm](https://cloud.githubusercontent.com/assets/9525062/8603876/e2310e4c-2648-11e5-87f9-1c6c13a16310.png)


Dont know why editing from the website made "284 additions and 284 deletions" when just modifying a line.


Done, new commit coming.


We discussed this out-of-band and clarified `wp_len`'s usage.  I think the LHS should be smaller because that's the quantity of bytes you're actually posting on the wire via USD (the padding region never gets copied/transmitted).


Removing the `defer` here actually has some side effect consequences. Specially for the inflight request logic. The defer means that the code inside it will only be executed when someone subscribes to it. And it will be executed every time someone subscribes to it.

With that we break the possibility of retrying on error using operators that subscribe again to retry and the inflight request check will break if the subscription occurs some time later than the creation of the Observable itself.

Why can't we just keep the `defer` here? Just escaping the function call like ``` `defer` ``` won't work?


[OpenBSD version](http://www.openbsd.org/cgi-bin/man.cgi/OpenBSD-current/man1/make.1?query=make&apropos=1#x5641524941424c452041535349474e4d454e5453) vs. [GNU version](https://www.gnu.org/software/make/manual/html_node/Automatic-Variables.html#index-_003c-_0028automatic-variable_0029) ..?


93469234-DemandResponse2 (vidanovic) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1690 of 1693 tests passed)

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


EDIT: Fixes #36


Do we need the pattern matching on q or would the dispatch on p be enough?

I would prefer a dispatch-only version without case classes and pattern matching. Pattern matching is not very coalgebraic and not idiomatic Java.

Even if we need  to special-case for some q values, I would prefer to use only dynamic dispatch:

~~~scala
trait Parser { p =>
    def alt(q) = q.alt2(p)
    def alt2(q) = new Parser {
        // default impl of alt here
    }
}

def fail(): Parser = new Parser {
    override def alt(q) = this
    override def alt2(q) = this
}
~~~


Done.


Please include a description of the error. You can test it by intentionally mangling one of the json files.


@raphasilvac that's the one I was using.


Before the rebase I had it with ``` `defer` ``` but now it complains about something I can't remember.

I don't use the `RxSwift` part and thus didn't now if it was a breaking change. I'll try to add it back and if I'm successful I'll commit to this same branch to keep all the code under the same `PR`


I was going to halfheartedly argue that there's no known way that we could run into such an issue...but salt moving things unexpectedly is the reason why any of the code you're reviewing exists in the first place. Addressed in the latest version.


I'll update ASAP


Ah, yeah, you're correct. Not sure why I thought a 100% map was 40x40 units, but fixed it now.


Oh, develop wouldn't compile because it was missing LanguageManager, and it
was just some basic platform things.

If you could get develop working then i'll branch off it next.

2015-07-09 20:31 GMT+02:00 James Liu <notifications@github.com>:

> Kuri, you might want to branch off of develop instead of master.
>
> Master is really only for completed/released code. develop is the active
> development branch, and will be merged into master only when ready for
> release
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/HouraiTeahouse/GensoShojokosen/commit/6869157ea8961efff2b71f6abee2ad0abb80f88c#commitcomment-12090972>
> .
>


I've been meaning to ask about that. The browser prefixes are generally for older browser support, and in this particular case, it was to fix a bug in Chrome because something else was setting the `webkitTransform` which apparently has a higher precedence than the standard `transform`.

For Material, do we recommend using all of the prefixes? If so, is this something that we could use an existing utility to provide, or should we make a utility for this?


Foarte bine!


@netj :) Thanks for the pointer! I was just got surprised by this piece of code when I see an `if` in a parser
(my knowledge on parsing does not get refreshed since ocaml :) my bad)


In above comment, `s/this/q/g`. Cannot edit comments with Github's Android app?!


The following comment is just an FYI (not something that needs to be change necessarily).

Lines 56-71 are reading all of the JSON files and creating a union of them all in `data`.  I figured that out pretty quickly, but was thrown for just a second by line 56 -  I thought, why create a string with an open brace?. Anyway, methods, functions, subroutines - these aren't just for encapsulating reusable chunks of program logic; they can also be used merely to express intent and improve the readability of the code. I'm oversimplifying here, but consider something like this alternative
```
data = unionAllDataFiles(dataset_dir)
PARSED_DATA = parseTableJson(data)
```
Clearly expressing intent in code is always a challenge.  Comments can help, but there's a popular school of thought that holds that they are often just a crutch and sometimes amount to no more than visual clutter.  When you get a chance, I suggest you checkout [Clean Code](http://www.amazon.com/Clean-Code-Handbook-Software-Craftsmanship/dp/0132350882).  It's chock full of practical advice and examples for clarifying intent, and, like the title suggest, writing clean code. The semantics can vary from one language to another, but the principles in Clean Code are fairly universal.


Formatting is off for this whole function.


93469234-DemandResponse2 (vidanovic) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


No worries.  I think this is still viewed as unconventional, but it seems people are gravitating towards merging lexer into parser especially when using powerful "parser combinators" like the ones in Scala.


> trying to use perlcritic again to be "best practice" burned me

quick question - can you point me to something in perlcritic that mentioned this, so I can look to see if it can/should be amended? many thanks!


Here's a heatmap of the infectious population.
![infectious](https://cloud.githubusercontent.com/assets/4325816/8604128/bd791e3c-2639-11e5-9329-5e7171a9c922.png)


Hi Christian, good to see you here. We love the work you've done on the Babbler project.

Regarding your points:

The userCreation is a test plugin used by devs to generate dummy users, vcards and roster entries on an Openfire system.

I'm one of the maintainers on the monitoring plugin and have a need to go through some PRs against it. To that end I thought it useful to be able to quickly setup OF with a set number of dummy users, roster entries and chat messages. The userCreation plugin already does two out of three, I don't think generating dummy chat data between those roster entries is that ill suited to its purpose.

I could move this code to the monitoring plugin but I'd rather avoid adding testing facilities to a popular production plugin. That I feel is a role better suited for the userCreation plugin whose purpose is after all to generate dummy test data.

I agree, a unit test would be excellent, and I thought about it but I wanted a way to be able to verify against MySQL, Postgres, SQL Server, etc. so testing live is the only option.


`$poz` este de fapt `$j`, care este `$i + 1` sau mai mult. Comparatia `$i  < $poz` va fi TRUE de fiecare data. Ceea ce inseamna ca nu este nevoie de ea.


And here's one showing the susceptible population.
![susc](https://cloud.githubusercontent.com/assets/4325816/8604151/eed6dd20-2639-11e5-8fa3-8975a165d05d.png)


Needs a newline at the end of the file.


Nice


I think the susceptible one is maybe easier to connect with the trap regions of figure 2.  High quality habitat is better in cells where either both high and low quality are white (no susceptibles), or when high quality is black (no infecteds).


I am not sure where to put the `versionadded`. Is it ok like that ?


i know i forgot to change that back in this commit, it almost works with this setup but there's a couple of packages, mostly gstreamer that are not whitelisted. you are right that testing install_dependencies makes sense. i was mostly trying if using ccache made a big difference in the times, i guess it does but it's pretty quick now as it is anyway


Ah ok :-) I didn't know that the userCreation plugin is just a test plugin for devs. I thought it's production plugin for non-devs. Then go on with it there! As you said, it's even better than then in the monitoring plugin.


To me as well.

I don't love having to resort to global options however I think this is so
far down the call chain that there is really no other way.

On Thu, Jul 9, 2015 at 2:14 PM, timelyportfolio <notifications@github.com>
wrote:

> Looks good to me.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ramnathv/htmlwidgets/commit/67d94f2eed83b1e1ccf6c146189a165ef630f81c#commitcomment-12090664>
> .
>


There is no link available for `django.template.base.Template.stream`, should I let it ?


I agree.

P

------------------------------------------
Paul Chafee Cross
2327 Univ. Way, Suite 2
Bozeman MT 59715
406 994 6908
http://www.nrmsc.usgs.gov/staff/pcross
-------------------------------------------

On Thu, Jul 9, 2015 at 12:59 PM, Clint Leach <notifications@github.com>
wrote:

> I think the susceptible one is maybe easier to connect with the trap
> regions of figure 2. High quality habitat is better in cells where either
> both high and low quality are white (no susceptibles), or when high quality
> is black (no infecteds).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/clint-leach/Metapop-Disease/commit/9cbfc84255f83fecba321fc4b204e9bd1ec43693#commitcomment-12091472>
> .
>


> xCard now has an object for game output.

Meaning that, Python's `print()` functionality is not directly used for xCard output.


Sorry. Very poor choice of words on my part.

Yes, the Firefox code has 5 booleans for all entries, and I'm assuming as well that it is not something required and can be dropped. Is the code in lines 41-55 the correct place to look? If so and I'm looking at it correctly, the fist "false" boolean needs to be removed (it would refer to "bool apponly;" in Firefox, which deals with binary components in app locations (N/A to Pale Moon)).


Hey mate,

Thanks a ton! I really appreciate the reply!

On Thu, Jul 9, 2015 at 8:59 AM, Ullrich Schäfer <notifications@github.com>
wrote:

> I wasn't planning to. It's been a long time since I worked on this the
> last time. Also I haven't been using much Growl myself since upgrading to
> Yosemite.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stigi/MountainGrowl/commit/2b956165685acc4c8d70c9096f7578fb04594f86#commitcomment-12076361>
> .
>


These longevities should be the same as in figures 1 and 2, since this is just a subset of those results.  Low longevity does correspond to a completely healthy scenario here, so I should point that out.


Make sure you have the dependencies listed in the readme. If components are missing even afterward, close out of Unity without saving anything and restart the project. If it still doesn't compile after that, log an issue.


well, it is perfectly OK to set a time "in the past" using timer_set_absolute, because of the overflow.


thanks :)


salam


I couldn't use it in Main class so I set the class to public. I can move the Main class in the ballot package and re-set the BallotProperties to private


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/17720/testReport/) for this commit**


Great! :)


I think this just changes the package name, not the package in which the rpm is generated -- can you elaborate on why this was necessary? Also, if possible, it should at least be presto-server, since it changes the name of the installed rpm to presto-server-rpm which is a bit redundant.


Yes that sounds like a good trade off! :)

> On 09 Jul 2015, at 21:04, Mattia <notifications@github.com> wrote:
>
> I couldn't use it in Main class so I set the class to public. I can move the Main class in the ballot package and re-set the BallotProperties to private
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/pdeboer/PPLibBallotConnector/commit/dfce9ae3f57417acdc3ff2136d28a032040cdc47#commitcomment-12091571>.
>


please init some move_info object with value, which can be consider as mate, and return it


This never threw an exception, right? Hilarious.


Clone, makes changes, and commit.


vzcxvcx


## This is a markdown file


Абсолютно не о чём не говорит название. Что проверяем то?


Thanks, fixed :)


`!check`


like you said , check here if we have evaluated this position and the evalinfo.depth is greater than current depth.
(also fix it for blacks)


can you please have a look at this?
`filter(Q(groups__name='Staff')|Q(groups__isnull=True))` before the annotation results in a list that only contains staff users. shouldn't there at least be all users who are either staff or nothing?
and i don't really know what the second solution is supposed to look like.


If it is broken now (I didn't test that commit yet), please go ahead and make a PR, since this one has been acepted, and I cant change it now.


Опять же, название неговорящее. Кстати, важный аспект - название метода должно быть глаголом.


@atavism FYI this was necessary to actually copy the new screenshots to the dist directory.


mTelephonyManager


Кстати почти все твои названия не соответствуют этому правилу :)


Call this user 'Player 1'?


And this "player 2"?


it's equivalent to `i < NaN` which is always false :p


Зачем теперь нужна эта переменная?


We were relying on this option, why was it removed?  Can it still work?


oh javascript...


`coordinates`


ah. right.


Собственно всё как я и предполагал - `SearchingCoordinates` делает практически то же самое, что и `Check`, их явно надо сливать в один


I think this made it in there by mistake. What is the "concept_child_relations" table?


кстати цикл с постусловием + `if` выше отлично сворачиваются в один цикл с предусловием :)


@hrydgard I am sorry for it, I just wanted you to know about this , it is only enabled hardware transform ...


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


Nice, thanks!


@digitarald @nickdesaulniers all right, I made the change in a branch ( https://github.com/texthtml/torch/tree/no-ua-sniffing ) but unfortunately I don't have a FirefoxOS device to test it on anymore :/ can one of you confirm that it's still working before I update it on the marketplace ?


It was primarily for my use but I know how useful this feature will be, it will be a PR.


I didn't realize trustedIdentity was there from the initial commit -- why not just "identity"?


к слову это называется field, board, возможно даже grid, но никак не blank :)


Hm, this should not happen. Did you get MKSH_NO_SIGSUSPEND defined somehow? If so, how? Does kLIBC define __KLIBC__ like Linux klibc does? (If so, ouch…)


There were a lot of urls not in the issues db, I think we might have to run that again/run the heroku scheduler to get all issues


Is there more to this sentence above?  `


I'm currently returning All the significant words ordered by frequency. I think it would be best to limit the results to like ~5-10 top words!


This task should handle the template files during development, copying templates to www.. I'm not sure about that end event, that doesn't do anything.

There's a problem with the ionicGardenTemplates at app.js...

I'm not sure how we could fix that, probably using an empty module during development, an generated module of course.


@EddyVerbruggen How come this release did not get pushed to the Telerik repository?


Кстати заливать бинари в репозиторий - дурной тон, добавь их в .gitignore


I am not sure where to place the `versionadded`. Is it ok like that ?


Discussed out-of-band and should only remove it for send.


fast-forwarding master to auto = cedf2f9b


@emmenko We should not call `fetch` here in order to allow to use `process` for downloading search results in junks.


@magnumripper This made mssql12 fail self test on CircleCI build.


let's avoid a merge conflict


As far as I know there is no other way to force update computed properties. Though I have a limited understanding of Emberjs and you should take my opinion with a grain of salt.


93469234-DemandResponse2 (vidanovic) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (651 of 651 tests passed)

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


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 125](https://teamcity.labkey.org/viewLog.html?buildId=260039&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:35


#473  is solved here fully.
We can create, rename and delete sections


%EASTER +1 ?
Don't we need to update also BE country ? Or it's logical to have two different dates.


I couldn't figure out how to do sync tasks with gulp, since everything is async..

I think the better solution probably will be something like this http://davidwalsh.name/gulp-run-sequence


@Sayantan2048 good job! all 4 formats now pass self test on Travis.


@jtreminio You are the man!!  :+1:


added riot tag language syntax for riotjs


I think 2001-10-01:00:00:00Z/etc should do too


Actually, it's both. Take a look in `.m2/repository/com/facebook/presto/presto` before this commit was applied and you'll see that the rpm was generated there instead of in `presto/presto-server-rpm` which breaks all the signatures when you try to closing the Maven staging repo


I took a totally different approach to the rotation function. It should be a lot more efficient now.


Fixes #39

Users with a preexisting post table need to add tn_w, tn_h to their SQL tables.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.403.94ce61c40440c1a08e42d56dc316d48ca0bc174f](http://build.palaso.org/viewLog.html?buildId=59529&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3319, ignored: 130 Build time: 00:08:10


I use to commit these files, once committed they rarely change, but you don't need to rebuild these images on every build and this is something I like.

But fuck off my opinion, we should ask some developers to hear their opinion


Okay, but see, I already pulled a request that fixed this same issue. What do now? :0


<y>


Ok so rethink this assignment.

u are connecting the posts and summaries, so migrations are needed. (ie foreign key etc.) and ur summaries needs to be nested inside posts.

all actions but the index action need to be filled out in summaries controller.


Hallo Orje,

1. Die Motoren haben keine Bremse. Die Pins auf dem Motor-Shield heißen aber "brake". Und wenn diese auf HIGH gesetzt werden, wird "gebremst" (der Motor ausgeschaltet).
2. Das Design des Roboters und auch das Listing sind nicht von mir. Im normalen Tageslicht ist das mit dem IR kaum ein Problem. Ein Ultraschall-Sensor wäre genauso denkbar. Hintergrund war vermutlich, den Roboter so günstig wie möglich für Einsteiger zu bekommen: Ultraschallsensoren sind i.d.R. teurer als IR-Sensoren.


@mikekirchem
Can you try if https://github.com/tpope/vim-fugitive/pull/649 helps?


Interesting. Proving that store(ptr+i) is covered by check(ptr, 4000) still requires induction variable analysis, so these hints wouldn't be a major simplification, but they would work in cases where the implementation can't automatically hoist checks due to side effects inside the loop.

On the other hand, such hints will add some overhead for implementations that don't otherwise have a bounds checking cost, and implementations that don't do any induction variable analysis.

I'd be inclined to categorize this as a FutureFeature. Do you agree?


The third argument is a default value, and it is a pointer. I would use `NULL`.


93469234-DemandResponse2 (vidanovic) - i386-Windows-7-VisualStudio-12: OK (1690 of 1693 tests passed)

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


The third argument is a default value, and it is a pointer. I would use NULL.


The idea of explicit `check` operations keeps coming up; I guess what we need to see before including this is a demonstration of a significant category of optimizations where the engine wouldn't have been able to achieve the same optimization by the normal bounds-check optimizations, but with the `check` inserted, the engine is able to prove safety.  I can imagine cases, I'd just like to see it in practice.  Because there seems to be a very real risk that overzealous insertion of checks could make things *slower* (viz., if they aren't found to be redundant with the internal checks inside loads/stores (think baseline compiler)).


I am sorry, I am missing why you changed this.

RFC 2661 indicates that the result code values for CDN messages, there will be 11 results.


Единообразие


ок


You are of course right, thanks!


What is the reason for such a funny decision for grid vs list view?


How about inspecting fn for the file extension?


> I intend to follow up by making the Dir.glob call even more broad, and let the post-processing filter take care of reducing even further.

@flarnie and I were chatting about this some the other day and what you outline here sounds like what we were thinking of as well. When thinking about good solutions, you might also want to keep #27 in mind, which is also a node.js convention.

You should probably add a test for this new behavior.

Also, if you add "Fixes #26" to your commit message, GitHub will close the issue for you once it is merged.


Thanks Bjorn. Note that there's some code here that I think belongs on https://github.com/cucumber/cucumber-ruby-core/blob/master/lib/cucumber/core/ast/location.rb. You should be able to use `Location.of_caller` for example, and I think the code to split up the path should also be in that class, maybe in another factory method.


RadianceRefactor (rpg777) - Win64-Windows-7-VisualStudio-12: OK (2174 of 2202 tests passed)

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


You could potentially avoid the memory overhead of the map by exploiting the sorted-ness of the original dictionary and using Arrays.binarySearch() instead.

Also, toUpperCase should specify a locale otherwise you could have the infamous Turkish/Azerbaijani "I" problem where "i" becomes "İ" instead of "I".


@frgm, commit message is worded poorly. OpenBSD version has it, but:
> <
> The name of the prerequisite from which this target is to be built, if a valid inference rule (suffix rule) is in scope.

while `gmake` defines this variable for rules of all kinds (even if there is no inference). The exact error message was:
```
Using $< in a non-suffix rule context is a GNUmake idiom
```


api_name , api_getmes ,api_title, api_detail are most likely ship names or armaments , get message when it drops or uograde, quest title and decription


hat Travis kein iojs 2.2.1 (was wir in den Devs verwenden)


This should just be an "else" clause on the above "if".


This is fantastic!


Исправил


Ah, I just want to catch the "overhead" that timer_periph might introduce which might cause setting a time "in the past". But just catching "now-value<0" wouldn't work, e.g., with now==2^32-1 and value==1000.


where is this load data Buffer


tun wir das? ich habe 1.6.4.
das coole bei travis ist wir können mehrere node und iojs versionen verwenden und er wird gegen alle testen. ist füge später noch die aktuellste iojs version hinzu…


flash bw tester should be manually started here


Сделал


Also reported on Google Play (Android 5.0 - Elephone P7000)


Option to mark differences.


You can do another PR, fixing the mistake.


TeamCity LabkeyRelease15.2 :: CDS postgres 9.4 [Build 4](https://teamcity.labkey.org/viewLog.html?buildId=260037&buildTypeId=LabkeyRelease152_CdsPostgres93) outcome was **SUCCESS**
Summary: Tests passed: 25 Build time: 00:34:33


Ошибочка, на автомате ввел. Исправил, по другому получаю Корзину


I was thinking of the case if somebody wanted to shift the distribution to a one based one (like some other implementation do), then you could simply have zero chance of hitting zero, and the distribution acts as if zero is not in the support. The same should apply from the other end, although I don't know of any use case of having trailing zeros in the probability vector, so I didn't implement it. Although, for consistency sake, I maybe should have.


reset _pos_value here, otherwise it would be added to the value of prev position?

_pos_value = 0;


Ah ok. Thanks for the fix, by the way!


Thank you! :smile:


Moving this line outside of the 'with' block ensures that the file is completely written before executing this step.


ESo no lo habia subido ud.

2015-07-09 0:08 GMT-05:00 Aslhey <notifications@github.com>:

> ¿Cómo asi, usted volvió a subir lo que yo subí anoche jaja?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rizotas/petR/commit/3e3aae55d14bd7160e51dde98bbab0d46ea20a3e#commitcomment-12076848>
> .
>


You may want to check out [`git-stats`](https://github.com/IonicaBizau/git-stats) and [`cli-github`]((https://github.com/IonicaBizau/cli-github). :dizzy:


The ALTER TABLE statements to create executions_idx3 are incorrect for all 3 databases - the index cannot be UNIQUE because it will prevent adding executions if there are multiple platforms and/or builds present.

The "create tables" scripts, however, are all correct.


@mjgiarlo This here's a doozie, but it's the only way I see to even vaguely satisfy the current code.  Sufficient cause to consider reimagining this behavior?


`yojson` should be propagated. See `META` file.


smoochies.


I will switch these two conditions around, since `result.isEmpty()` is much quicker to check and if false `implicitiBaseFromProject()` will not be evaluated. I will do this change after I merge.


*I can't do it right now